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

Don't eliminate frame pointer at -O0 #2480

Closed
JohanEngelen opened this issue Jan 3, 2018 · 3 comments
Closed

Don't eliminate frame pointer at -O0 #2480

JohanEngelen opened this issue Jan 3, 2018 · 3 comments

Comments

@JohanEngelen
Copy link
Member

For previous discussion on this topic: #2096

I think we should not eliminate the frame pointer at -O0. It is not just our own stack traces that are (no longer) affected. ASan also does not show correct/complete stack traces with frame pointer elimination.

Clang does not even eliminate the frame pointer at -O3 on macOS. We don't have to mimic that, but at -O0 I feel we really should keep the fp.

@dnadlinger
Copy link
Member

dnadlinger commented Jan 4, 2018

I agree, we shouldn't do frame-pointer elimination at -O0. (Is this possibly a consequence of a LLVM API change at some point?) Just to recap, in #2096 I was merely arguing for a better backtrace mechanism, rather than against disabling fp elim.

@JohanEngelen
Copy link
Member Author

Related: #2222

@kinke
Copy link
Member

kinke commented Jan 13, 2018

Fixed by #2483.

@kinke kinke closed this as completed Jan 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants