Author: Lang Hames Date: 2023-08-23T17:19:07-07:00 New Revision: 8a62d6ba7edc3a7d397e52884a9ce63b4e579ae1
URL: https://github.com/llvm/llvm-project/commit/8a62d6ba7edc3a7d397e52884a9ce63b4e579ae1 DIFF: https://github.com/llvm/llvm-project/commit/8a62d6ba7edc3a7d397e52884a9ce63b4e579ae1.diff LOG: [ORC][clang-repl] Fix UnitTest after 122ebe3b500. Commit 122ebe3b500 changed the way that we look up eh-frame registration functions. This made LLJIT work "out of the box" in some Linux configs that didn't work before, but caused InterpreterExceptionTest to start failing: The test was skipped if a construction of a basic LLJIT config failed, but the test actually depended on debugger support working too. When the basic config started working we no longer skipped the test, then failed due to missing debugger support. I've extended the skip-test check to include the debugger support requirement, which should fix the issue. Added: Modified: clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp Removed: ################################################################################ diff --git a/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp b/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp index 70e10b1e53bd9d..8700f506d9b17d 100644 --- a/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp +++ b/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp @@ -52,7 +52,9 @@ TEST(InterpreterTest, CatchException) { llvm::InitializeNativeTargetAsmPrinter(); { - auto J = llvm::orc::LLJITBuilder().create(); + auto J = llvm::orc::LLJITBuilder() + .setEnableDebuggerSupport(true) + .create(); if (!J) { // The platform does not support JITs. // Using llvm::consumeError will require typeinfo for ErrorInfoBase, we _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits