This is an archive of the discontinued LLVM Phabricator instance.

Add GCC's noexcept-type alias for c++1z-compat-mangling
ClosedPublic

Authored by teemperor on Jun 21 2017, 3:31 AM.

Diff Detail

Event Timeline

teemperor created this revision.Jun 21 2017, 3:31 AM

I didn't know gcc had its own option. This change seems reasonable to me.

Since c++1z-compat-mangling was added just a few days ago, should we just rename it instead of adding an alias?

@ahatanak I think we can leave the more expressive clang name for this warning and just add the bit cryptic GCC name for compability. But I don't have a strong opinion on this.

For the record, here's what GCC does (from https://gcc.gnu.org/onlinedocs/gcc/C_002b_002b-Dialect-Options.html#C_002b_002b-Dialect-Options which probably has a typo, the second -Wnoexcept is likely meant to be -Wnoexcept-type): "Enabled by -Wabi and -Wc++1z-compat." -Wc++1z-compat is probably meant to enable a group out of which -Wnoexcept-type is just one (currently the only one as far as I can see). FWIW, we notice the missing -Wno-noexcept-type but not -Wc++1z-compat.

ping. Any objections to adding this GCC alias?

This revision is now accepted and ready to land.Jul 18 2017, 5:23 AM
teemperor closed this revision.Jul 18 2017, 11:53 AM