-
Notifications
You must be signed in to change notification settings - Fork 2k
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
native: use -g3 instead of -g #10877
Conversation
This usually increases the size of the debug symbols significantly, but on the other hand we already use it on arm since a long time ago. The major difference is that all preprocessor macros are included in the debug table. |
I think size isn't really an issue on |
(especially since we don't compile with debug symbols per default [edit: with |
👍 I was mostly thinking of the CI disk usage. |
Rebased to current master. |
I used I could run without problem
I did not test the new debug output as I do not really know how to do it. I cannot compile The issue with the size is not important here. |
From man gcc:
So it make sense to enable it for native. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If somebody finds any issue with this in the future, it could be changed back but it makes sense to help debugging.
Thanks @cladmi |
Contribution description
I use
CFLAGS=-g3
for years and it is far more useful than the provided-g
in most of ournative
-specific targets. I finally got fed up with it enough to try to change it ;-).Testing procedure
The affected targets should still be able to compile an application:
all-valgrind
all-debug
all-cachegrind
all-asan
The application should still be running without any problems.
Issues/PRs references
#4413 I guess, but this only focuses on
native