[llvm] [GlobalOpt] Use cast instead of dyn_cast. NFC (PR #144634)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 17 21:56:49 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Craig Topper (topperc)
<details>
<summary>Changes</summary>
The dyn_cast was not checked for null, and the cast is guaranteed to succeed by an earlier check.
---
Full diff: https://github.com/llvm/llvm-project/pull/144634.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/IPO/GlobalOpt.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Transforms/IPO/GlobalOpt.cpp b/llvm/lib/Transforms/IPO/GlobalOpt.cpp
index 7db0586386506..4a06e0fa619c0 100644
--- a/llvm/lib/Transforms/IPO/GlobalOpt.cpp
+++ b/llvm/lib/Transforms/IPO/GlobalOpt.cpp
@@ -2169,7 +2169,7 @@ static bool tryWidenGlobalArraysUsedByMemcpy(
unsigned NumBytesToCopy = BytesToCopyOp->getZExtValue();
- auto *Alloca = dyn_cast<AllocaInst>(CI->getArgOperand(0));
+ auto *Alloca = cast<AllocaInst>(CI->getArgOperand(0));
uint64_t DZSize = Alloca->getAllocatedType()->getArrayNumElements();
uint64_t SZSize = SourceDataArray->getType()->getNumElements();
unsigned ElementByteWidth = SourceDataArray->getElementByteSize();
``````````
</details>
https://github.com/llvm/llvm-project/pull/144634
More information about the llvm-commits
mailing list