vmiklos added a comment.

In https://reviews.llvm.org/D32945#748225, @malcolm.parsons wrote:

> See uses of `OptionsView::getLocalOrGlobal()`.


Ah, I see. Then yes, I think it makes sense; e.g. for the above cppunit case 
ideally I want to avoid any kind of warning from macros, since none of them can 
be easily fixed in system headers. (And the other way around, if system headers 
are not a problem, then all kind of warnings inside macros are probably 
interesting.) The updated patch uses `getLocalOrGlobal()` for the new option.


https://reviews.llvm.org/D32945



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

Reply via email to