Skip to content
This repository was archived by the owner on Feb 13, 2025. It is now read-only.

[2.7] Protect climb_stack_and_eval_frame from trivialization #227

Merged
merged 2 commits into from
Aug 23, 2019
Merged

[2.7] Protect climb_stack_and_eval_frame from trivialization #227

merged 2 commits into from
Aug 23, 2019

Conversation

mmirate
Copy link

@mmirate mmirate commented Aug 21, 2019

gcc-4.8.5 (and possibly other versions between 4.7 and 5.4) trivializes both instances of the side-effect-only alloca trick: not only climb_stack_and_transfer, but also climb_stack_and_eval_frame.

Thus the same global-write trick must be used both there and here.

(This is of course entirely inapplicable to 3.x and master, due to #163.)

gcc >= 4.7 trivializes not only climb_stack_and_transfer,
but also climb_stack_and_eval_frame.

Thus the same global-write trick must be used both there
and here.
Copy link

@akruis akruis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thank you for reporting this issue.

@akruis akruis merged commit 836b2c4 into stackless-dev:2.7-slp Aug 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants