[clang-repl] Disable out of process JIT tests on non-unix platforms

[kr-2003]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Abhinav Kumar (kr-2003)

Changes

---

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

1 Files Affected:

• (modified) clang/unittests/Interpreter/CMakeLists.txt (+2-2)

diff --git a/clang/unittests/Interpreter/CMakeLists.txt b/clang/unittests/Interpreter/CMakeLists.txt
index 7b8dcfc9b0546..66b396b53cb55 100644
--- a/clang/unittests/Interpreter/CMakeLists.txt
+++ b/clang/unittests/Interpreter/CMakeLists.txt
@@ -37,7 +37,7 @@ set(CLANG_REPL_TEST_SOURCES
   CodeCompletionTest.cpp
 )
 
-if(TARGET compiler-rt)
+if(TARGET compiler-rt AND LLVM_ON_UNIX)
   list(APPEND CLANG_REPL_TEST_SOURCES
     OutOfProcessInterpreterTests.cpp
   )
@@ -61,7 +61,7 @@ add_distinct_clang_unittest(ClangReplInterpreterTests
   ${LLVM_COMPONENTS_TO_LINK}
   )
 
-if(TARGET compiler-rt)
+if(TARGET compiler-rt AND LLVM_ON_UNIX)
   add_dependencies(ClangReplInterpreterTests 
     llvm-jitlink-executor 
     compiler-rt

[kr-2003]

@OCHyams Can you review this?

[OCHyams]

LGTM to get the build fixed (works locally for me)

I am not familiar with the test or what it's testing, so I can't weigh in as to whether there should also be a windows version of the test.

Thanks for the speedy fix.

[anutosh491]

Hey @OCHyams

Thanks for the review. Yes we don't expect this to work on Windows !