Skip to content

Conversation

@MSLaguana
Copy link
Contributor

The code to deal with x86 aligned return values was too loose, occasionally resulting in missing stack frames. This change tries to more accurately determine when the return value has shifted due to alignment.

Copy link
Contributor

@boingoing boingoing left a comment

Choose a reason for hiding this comment

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

Looks fine to me. I'd like @LouisLaf to weigh in, too.

Thanks for fixing this 👍

Copy link
Collaborator

@LouisLaf LouisLaf left a comment

Choose a reason for hiding this comment

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

:shipit:

@chakrabot chakrabot merged commit 340a4bb into chakra-core:release/1.7 Nov 16, 2017
chakrabot pushed a commit that referenced this pull request Nov 16, 2017
Merge pull request #4252 from MSLaguana:fixStackWalk

The code to deal with x86 aligned return values was too loose, occasionally resulting in missing stack frames. This change tries to more accurately determine when the return value has shifted due to alignment.
chakrabot pushed a commit that referenced this pull request Nov 18, 2017
Merge pull request #4252 from MSLaguana:fixStackWalk

The code to deal with x86 aligned return values was too loose, occasionally resulting in missing stack frames. This change tries to more accurately determine when the return value has shifted due to alignment.
chakrabot pushed a commit that referenced this pull request Nov 18, 2017
…ing issue.

Merge pull request #4252 from MSLaguana:fixStackWalk

The code to deal with x86 aligned return values was too loose, occasionally resulting in missing stack frames. This change tries to more accurately determine when the return value has shifted due to alignment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants