Add fixmes

Author: mcbarton

clang/unittests/Interpreter/CodeCompletionTest.cpp

@@ -29,6 +29,10 @@ class CodeCompletionTest : public InterpreterTestBase {
   std::unique_ptr<clang::Interpreter> Interp;
 
   void SetUp() override {
+#FIXME : WebAssembly doesn't currently support Jit (see
+#https: // github.com/llvm/llvm-project/pull/150977#discussion_r2237521095).
+#so this check of HostSupportsJIT has been skipped
+#over until support is added, and HostSupportsJIT can return true.
 #ifndef __EMSCRIPTEN__
     if (!HostSupportsJIT())
       GTEST_SKIP();

clang/unittests/Interpreter/IncrementalCompilerBuilderTest.cpp

@@ -37,6 +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
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif

clang/unittests/Interpreter/InterpreterExtensionsTest.cpp

@@ -75,6 +75,10 @@ struct OutOfProcInterpreter : public Interpreter {
 };
 
 TEST_F(InterpreterExtensionsTest, FindRuntimeInterface) {
+#FIXME : WebAssembly doesn't currently support Jit (see
+#https: // github.com/llvm/llvm-project/pull/150977#discussion_r2237521095).
+#so this check of HostSupportsJIT has been skipped
+#over until support is added, and HostSupportsJIT can return true.
 #ifndef __EMSCRIPTEN__
   if (!HostSupportsJIT())
     GTEST_SKIP();

clang/unittests/Interpreter/InterpreterTest.cpp

@@ -147,6 +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
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif
@@ -259,6 +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
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif
@@ -301,6 +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
 #ifdef __EMSCRIPTEN__
   GTEST_SKIP() << "Test fails for Emscipten builds";
 #endif

clang/unittests/Interpreter/InterpreterTestFixture.h

@@ -38,6 +38,10 @@ class InterpreterTestBase : public ::testing::Test {
   }
 
   void SetUp() override {
+#FIXME : WebAssembly doesn't currently support Jit (see
+#https: // github.com/llvm/llvm-project/pull/150977#discussion_r2237521095).
+#so this check of HostSupportsJIT has been skipped
+#over until support is added, and HostSupportsJIT can return true.
 #ifndef __EMSCRIPTEN__
     if (!HostSupportsJIT())
       GTEST_SKIP();