[PATCH] D34251: Add a new driver option to disable warning about c++17's non-throwing exception specification in function signature

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 19 16:25:31 PDT 2017


dexonsmith added a comment.

> We use the `-Wc++NN-compat-` prefix on all the other subwarnings of `-Wc++NN-compat` warnings

Interesting; I thought I saw a counter-example recently, and took that as the rule (although I can't for the life of me remember what it was).  `-Wc++NN-compat-mangling` SGTM though.

> Just curious, though: what is the motivation for enabling the rest of `-Wc++1z-compat` but not this part? This seems a lot more serious than the other warnings in the group. (Regardless, I think this patch is useful for the opposite: enabling just this one warning without the rest...)

A user is recompiling the codebase as a unit so binary compatibility doesn't matter, but language semantics do.


https://reviews.llvm.org/D34251





More information about the cfe-commits mailing list