[clang] [libc++] Re-enable the clang_modules_include test for Objective-C++ (PR #66801)

via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 19 11:49:26 PDT 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

<details>
<summary>Changes</summary>

This reverts commit aa60b2687, which was a temporary workaround.

This was originally https://reviews.llvm.org/D158694.

---
Full diff: https://github.com/llvm/llvm-project/pull/66801.diff


2 Files Affected:

- (added) clang/foo () 
- (modified) libcxx/test/libcxx/clang_modules_include.gen.py (-3) 


``````````diff
diff --git a/clang/foo b/clang/foo
new file mode 100644
index 000000000000000..e69de29bb2d1d64
diff --git a/libcxx/test/libcxx/clang_modules_include.gen.py b/libcxx/test/libcxx/clang_modules_include.gen.py
index b4c5c79a47f6823..b4aecfe5d8faae6 100644
--- a/libcxx/test/libcxx/clang_modules_include.gen.py
+++ b/libcxx/test/libcxx/clang_modules_include.gen.py
@@ -44,11 +44,8 @@
 #include <{header}>
 """)
 
-# TODO: Remove the UNSUPPORTED{BLOCKLIT}: clang-modules-build once issues with this test have been figured out.
 print(f"""\
 //--- __std_clang_module.compile.pass.mm
-// UNSUPPORTED{BLOCKLIT}: clang-modules-build
-
 // RUN{BLOCKLIT}: %{{cxx}} %s %{{flags}} %{{compile_flags}} -fmodules -fcxx-modules -fmodules-cache-path=%t -fsyntax-only
 
 // REQUIRES{BLOCKLIT}: clang-modules-build

``````````

</details>


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


More information about the cfe-commits mailing list