This is an archive of the discontinued LLVM Phabricator instance.

[Clang][Modules] Support `requires cplusplus20` in a modulemap
ClosedPublic

Authored by egorzhdan on May 4 2023, 7:16 AM.

Details

Summary

This change adds the support for requires cplusplus20 directive for a module declared in a Clang modulemap, similarly to existing directives requires cplusplus17, requires cplusplus14, etc.

rdar://108891417

Diff Detail

Event Timeline

egorzhdan created this revision.May 4 2023, 7:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 4 2023, 7:16 AM
egorzhdan requested review of this revision.May 4 2023, 7:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 4 2023, 7:16 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
bruno accepted this revision.May 4 2023, 10:44 AM
bruno added reviewers: Bigcheese, ChuanqiXu.

LGTM

This revision is now accepted and ready to land.May 4 2023, 10:44 AM

@Bigcheese, @ChuanqiXu, could you please take a look at this patch?

ChuanqiXu accepted this revision.May 15 2023, 3:35 AM

LGTM. Sorry for misleading.

egorzhdan closed this revision.Jun 6 2023, 3:03 AM

Looks like https://reviews.llvm.org/D150773 got there first.