[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