benlangmuir marked an inline comment as done. benlangmuir added inline comments.
================ Comment at: clang/include/clang/Basic/DiagnosticOptions.h:128 + /// whether -Wsystem-headers is enabled on a per-module basis. + std::vector<std::string> SystemHeaderWarningsModules; + ---------------- iana wrote: > benlangmuir wrote: > > jansvoboda11 wrote: > > > Out of interest, is there an existing use-case for having multiple > > > modules here, or is this just future-proofing? > > I'm not using it currently, but I think it makes sense to handle any > > number. You could have multiple such modules you're testing together. > I'd imagine you'd typically have Framework and Framework_Private in there Ah, good point. I am in fact doing that already and just forgot. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156948/new/ https://reviews.llvm.org/D156948 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits