[PATCH] D146986: Downgrade reserved module identifier error into a warning
Chuanqi Xu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 27 19:56:13 PDT 2023
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
More information about the cfe-commits
mailing list