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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to