Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing x86 stack walking issue. #4252

Merged
merged 1 commit into from
Nov 16, 2017

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