[PATCH] D28148: [Sema] Suppress warnings for C's zero initializer
S. Gilles via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 28 17:16:13 PST 2016
sgilles added a comment.
In https://reviews.llvm.org/D28148#632002, @rsmith wrote:
> Why are you adding a language option for this? Just use `!LangOpts.CPlusPlus`.
I didn't want to have this change accidentally apply to other, non-C++ languages, since I'm not sure which languages would go through this path (Objective C, perhaps). If that's not a problem, I'll remove the language option and switch the checks to `!LangOpts.CPlusPlus`, which should be much cleaner.
https://reviews.llvm.org/D28148
More information about the cfe-commits
mailing list