[PATCH] D150937: [clang-repl] Disable all tests on unsupported platforms

Vassil Vassilev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 21 23:24:58 PDT 2023


v.g.vassilev added a comment.

I would support this change but instead of asking the JIT we should just add `-fsyntax-only` to these invocations as they do not rely on the JIT at all.



================
Comment at: clang/test/Interpreter/incremental-mode.cpp:3
 // RUN: clang-repl -Xcc -emit-llvm 
+// UNSUPPORTED: system-aix
 // expected-no-diagnostics
----------------
This test does not run anything, so this should be supported. Perhaps we could add `-fsyntax-only` mode to the RUN lines above.


================
Comment at: clang/unittests/Interpreter/IncrementalProcessingTest.cpp:74
+    return;
   std::vector<const char *> ClangArgv = {"-Xclang", "-emit-llvm-only"};
   auto CI = llvm::cantFail(IncrementalCompilerBuilder::create(ClangArgv));
----------------
Likewise, here `-fsyntax-only` mode should be enough since we do not execute anything.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150937/new/

https://reviews.llvm.org/D150937



More information about the cfe-commits mailing list