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
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
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
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
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