[llvm] [NewGVN] Check intrinsic ID before accessing operands (PR #141571)
Mariusz Sikora via llvm-commits
llvm-commits at lists.llvm.org
Tue May 27 02:19:15 PDT 2025
https://github.com/mariusz-sikora-at-amd created https://github.com/llvm/llvm-project/pull/141571
This patch is fixing assertions in downstream tests
>From 1606215c998fb8e5dcb983ec6047107555700c18 Mon Sep 17 00:00:00 2001
From: Mariusz Sikora <mariusz.sikora at amd.com>
Date: Tue, 27 May 2025 05:16:46 -0400
Subject: [PATCH] [NewGVN] Check intrinsic ID before accessing operands
This patch is fixing assertions in downstream tests
---
llvm/lib/Transforms/Scalar/NewGVN.cpp | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/llvm/lib/Transforms/Scalar/NewGVN.cpp b/llvm/lib/Transforms/Scalar/NewGVN.cpp
index 85bd8ef1f09a8..584cb0e89ef66 100644
--- a/llvm/lib/Transforms/Scalar/NewGVN.cpp
+++ b/llvm/lib/Transforms/Scalar/NewGVN.cpp
@@ -1530,11 +1530,12 @@ NewGVN::performSymbolicLoadCoercion(Type *LoadType, Value *LoadPtr,
}
if (auto *II = dyn_cast<IntrinsicInst>(DepInst)) {
- auto *LifetimePtr = II->getOperand(1);
- if (II->getIntrinsicID() == Intrinsic::lifetime_start &&
- (LoadPtr == lookupOperandLeader(LifetimePtr) ||
- AA->isMustAlias(LoadPtr, LifetimePtr)))
- return createConstantExpression(UndefValue::get(LoadType));
+ if (II->getIntrinsicID() == Intrinsic::lifetime_start) {
+ auto *LifetimePtr = II->getOperand(1);
+ if (LoadPtr == lookupOperandLeader(LifetimePtr) ||
+ AA->isMustAlias(LoadPtr, LifetimePtr))
+ return createConstantExpression(UndefValue::get(LoadType));
+ }
}
// All of the below are only true if the loaded pointer is produced
More information about the llvm-commits
mailing list