ChuanqiXu accepted this revision. ChuanqiXu added a comment. This revision is now accepted and ready to land.
> Presumably adding an alias for #pragma clang system_header called > system_module wouldn't be too hard? (though the pragma is also being removed > from libc++ soon, I think, in favor of -isystem usage, so maybe that's a sign > the pragma's not a great way to do) Yeah, it shouldn't be hard but it looks not good too. I feel the current patch is better/simpler as a workaround. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146986/new/ https://reviews.llvm.org/D146986 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits