[libcxx-commits] [libcxx] [libc++][modules] Mark as implemented. (PR #90091)

[Nikolas Klauser via libcxx-commits]

https://github.com/philnik777 requested changes to this pull request.

IIRC part the reasoning for adding modules unconditionally was that vendors don't have to actually install them. Them change seems to me like it forces vendors to install them, since we claim support through the FTM. I'm also not sure we should already claim support given that, AFAICT, it's still very much an experimental feature which isn't supported in a lot of cases. Third and last: when we set the macro, should we also set it in C++20?

https://github.com/llvm/llvm-project/pull/90091