[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

2023-06-06 Thread Egor Zhdan via Phabricator via cfe-commits
egorzhdan closed this revision. egorzhdan added a comment. Looks like https://reviews.llvm.org/D150773 got there first. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149850/new/ https://reviews.llvm.org/D149850

[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

2023-05-15 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu accepted this revision. ChuanqiXu added a comment. LGTM. Sorry for misleading. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149850/new/ https://reviews.llvm.org/D149850 ___ cfe-commits mailing

[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

2023-05-15 Thread Egor Zhdan via Phabricator via cfe-commits
egorzhdan added a comment. @Bigcheese, @ChuanqiXu, could you please take a look at this patch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149850/new/ https://reviews.llvm.org/D149850 ___ cfe-commits m

[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

2023-05-04 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno accepted this revision. bruno added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149850/new/ https://reviews.llvm.org/D149850 ___ c

[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

2023-05-04 Thread Egor Zhdan via Phabricator via cfe-commits
egorzhdan created this revision. Herald added a project: All. egorzhdan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change adds the support for `requires cplusplus20` directive for a module declared in a Clang modulemap, similarly