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

tl;dr section suggests options that it maybe shouldn't #333

Closed
AaronBallman opened this issue Dec 6, 2023 · 2 comments · Fixed by #358
Closed

tl;dr section suggests options that it maybe shouldn't #333

AaronBallman opened this issue Dec 6, 2023 · 2 comments · Fixed by #358

Comments

@AaronBallman
Copy link

Clang doesn't support -Wtrampolines and will diagnose it as an unknown warning option, so it might not make sense to include it in the tl;dr section (I would expect some people will copy/paste that section without giving it much thought).

Also, _FORTIFY_SOURCE support differs between Clang and GCC (https://developers.redhat.com/blog/2020/02/11/toward-_fortify_source-parity-between-clang-and-gcc) and those differences should be called out explicitly in the tl;dr so people don’t get a false sense of security with the option.

@thomasnyman
Copy link
Contributor

Possibly partial duplicate of #277.

The feature parity issue is addressed by footnote referenced from the main table but I see that footnote seems to have gathered a couple of erroneous references, probably due to copy-pasting. Let's fix those and make sure it also is linked from the tl;dr;.

@david-a-wheeler
Copy link
Contributor

We now have a table showing "if condition X, use option Y". I think it's fine to have a gcc-specific entry (or a clang-specific entry) if an option is supported by one but not the other.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants