[llvm] [BOLT] Fix assertion test (PR #169635)

via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 26 03:03:18 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-bolt

Author: Gergely Bálint (bgergely0)

<details>
<summary>Changes</summary>

Add `#ifndef NDEBUG` to exclude the assertion test from a
no assertion build.

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


1 Files Affected:

- (modified) bolt/unittests/Core/MCPlusBuilder.cpp (+2) 


``````````diff
diff --git a/bolt/unittests/Core/MCPlusBuilder.cpp b/bolt/unittests/Core/MCPlusBuilder.cpp
index 439d72a343ce8..7f2a2100363e9 100644
--- a/bolt/unittests/Core/MCPlusBuilder.cpp
+++ b/bolt/unittests/Core/MCPlusBuilder.cpp
@@ -171,9 +171,11 @@ TEST_P(MCPlusBuilderTester, AArch64_BTI) {
   ASSERT_EQ(II->getOperand(0).getImm(), 34);
   ASSERT_TRUE(BC->MIB->isBTILandingPad(*II, true, false));
 
+#ifndef NDEBUG
   MCInst BTIinvalid;
   ASSERT_DEATH(BC->MIB->createBTI(BTIinvalid, false, false),
                "No target kinds!");
+#endif
 
   MCInst Paciasp = MCInstBuilder(AArch64::PACIASP);
   II = BB->addInstruction(Paciasp);

``````````

</details>


https://github.com/llvm/llvm-project/pull/169635


More information about the llvm-commits mailing list