[libcxx-commits] [PATCH] D157625: [libc++][modules] Simplifies C++20 module testing.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/test/libcxx/module_std.gen.py:130
 // UNSUPPORTED{BLOCKLIT}: c++03, c++11, c++14, c++17, c++20
+// UNSUPPORTED{BLOCKLIT}: c++26
+// UNSUPPORTED{BLOCKLIT}: libcpp-has-no-std-modules
----------------
The C++26 support should be enabled again in D153408


================
Comment at: libcxx/test/lit.local.cfg:42
+        # TODO MODULES test whether this is fixed with CMake 3.27.
+        quit()
+    elif std == "cxx23":
----------------
ldionne wrote:
> What is `quit()`?
This is a left over and doesn't work correctly. I'll remove it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157625