[clang] [llvm] Enable running ClangReplInterpreterTests in an Emscripten environment (PR #150977)

[via cfe-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff HEAD~1 HEAD --extensions h,cpp -- clang/unittests/Interpreter/CodeCompletionTest.cpp clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp clang/unittests/Interpreter/InterpreterExtensionsTest.cpp clang/unittests/Interpreter/InterpreterTest.cpp clang/unittests/Interpreter/InterpreterTestFixture.h
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp b/clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp
index 6f22e36ae..e3aeff367 100644
--- a/clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp
+++ b/clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp
@@ -37,10 +37,10 @@ TEST(IncrementalCompilerBuilder, SetCompilerArgs) {
 }
 
 TEST(IncrementalCompilerBuilder, SetTargetTriple) {
-# FIXME: This test doesn't current work for Emscripten builds.
-# It should be possible to make it work. For details on how it fails and
-# the current progress to enable this test see
-# the following Github issue https://github.com/llvm/llvm-project/issues/153461
+#FIXME : This test doesn't current work for Emscripten builds.
+#It should be possible to make it work.For details on how it fails and
+#the current progress to enable this test see
+#the following Github issue https: // github.com/llvm/llvm-project/issues/153461
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif
diff --git a/clang/unittests/Interpreter/InterpreterTest.cpp b/clang/unittests/Interpreter/InterpreterTest.cpp
index 3dd465084..c877b6bf0 100644
--- a/clang/unittests/Interpreter/InterpreterTest.cpp
+++ b/clang/unittests/Interpreter/InterpreterTest.cpp
@@ -147,10 +147,10 @@ TEST_F(InterpreterTest, DeclsAndStatements) {
 }
 
 TEST_F(InterpreterTest, UndoCommand) {
-# FIXME: This test doesn't current work for Emscripten builds.
-# It should be possible to make it work. For details on how it fails and
-# the current progress to enable this test see
-# the following Github issue https://github.com/llvm/llvm-project/issues/153461
+#FIXME : This test doesn't current work for Emscripten builds.
+#It should be possible to make it work.For details on how it fails and
+#the current progress to enable this test see
+#the following Github issue https: // github.com/llvm/llvm-project/issues/153461
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif
@@ -263,10 +263,10 @@ static NamedDecl *LookupSingleName(Interpreter &Interp, const char *Name) {
 }
 
 TEST_F(InterpreterTest, InstantiateTemplate) {
-# FIXME: This test doesn't current work for Emscripten builds.
-# It should be possible to make it work. For details on how it fails and
-# the current progress to enable this test see
-# the following Github issue https://github.com/llvm/llvm-project/issues/153461
+#FIXME : This test doesn't current work for Emscripten builds.
+#It should be possible to make it work.For details on how it fails and
+#the current progress to enable this test see
+#the following Github issue https: // github.com/llvm/llvm-project/issues/153461
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif
@@ -309,10 +309,10 @@ TEST_F(InterpreterTest, InstantiateTemplate) {
 }
 
 TEST_F(InterpreterTest, Value) {
-# FIXME: This test doesn't current work for Emscripten builds.
-# It should be possible to make it work. For details on how it fails and
-# the current progress to enable this test see
-# the following Github issue https://github.com/llvm/llvm-project/issues/153461
+#FIXME : This test doesn't current work for Emscripten builds.
+#It should be possible to make it work.For details on how it fails and
+#the current progress to enable this test see
+#the following Github issue https: // github.com/llvm/llvm-project/issues/153461
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif

``````````

</details>


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