[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