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