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

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 19 11:48:19 PDT 2023


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

This reverts commit aa60b2687, which was a temporary workaround.

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

>From 7683064a0b9aa7d76cf6fcbd8c804ad897c4b625 Mon Sep 17 00:00:00 2001
From: Louis Dionne <ldionne.2 at gmail.com>
Date: Wed, 23 Aug 2023 22:23:17 -0400
Subject: [PATCH] [libc++] Re-enable the clang_modules_include test for
 Objective-C++

This reverts commit aa60b2687, which was a temporary workaround.
---
 clang/foo                                       | 0
 libcxx/test/libcxx/clang_modules_include.gen.py | 3 ---
 2 files changed, 3 deletions(-)
 create mode 100644 clang/foo

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



More information about the cfe-commits mailing list