llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang <details> <summary>Changes</summary> This test fails as .eh_frame handling is not yet implemented for RISC-V in JITLink. #<!-- -->66067 is proposed to address this. Skip the test until the issue is resolved. It seems that D159167 enabled this test for more than just ppc64. As the test always failed, it just wasn't run until now, I think skipping is the correct interim approach (as is already done for Arm, Darwin, and others). --- Full diff: https://github.com/llvm/llvm-project/pull/68216.diff 1 Files Affected: - (modified) clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp (+5) ``````````diff diff --git a/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp b/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp index 2f1c4efb381f00b..7b47d93446192ba 100644 --- a/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp +++ b/clang/unittests/Interpreter/ExceptionTests/InterpreterExceptionTest.cpp @@ -122,6 +122,11 @@ extern "C" int throw_exception() { Triple.getArch() == llvm::Triple::aarch64_32)) GTEST_SKIP(); + // FIXME: RISC-V fails as .eh_frame handling is not yet implemented in + // JITLink for RISC-V. See PR #66067. + if (Triple.isRISCV()) + GTEST_SKIP(); + llvm::cantFail(Interp->ParseAndExecute(ExceptionCode)); testing::internal::CaptureStdout(); auto ThrowException = `````````` </details> https://github.com/llvm/llvm-project/pull/68216 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
