[PATCH] D122394: [C++20][Modules] Correct an assert for modules-ts.

Nathan Sidwell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 25 03:57:00 PDT 2022


urnathan accepted this revision.
urnathan added a comment.
This revision is now accepted and ready to land.

Understood.  To document the approach:

1. clearly separate -fmodules-ts semantics (which are vague in places, but whatever)
2. implement c++20 modules semantics

3a) switch -fmodules-ts option to behave as #2 and delete any now-unreachable TS-specific code
3b) deprecate -fmodules-ts option and then remove.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122394/new/

https://reviews.llvm.org/D122394



More information about the cfe-commits mailing list