[llvm] [llvm::transforms] Add overflow check in AllocaInst::getAllocationSize (PR #97170)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 1 02:31:50 PDT 2024
================
@@ -60,22 +61,29 @@ static cl::opt<bool> DisableI2pP2iOpt(
std::optional<TypeSize>
AllocaInst::getAllocationSize(const DataLayout &DL) const {
TypeSize Size = DL.getTypeAllocSize(getAllocatedType());
- if (isArrayAllocation()) {
- auto *C = dyn_cast<ConstantInt>(getArraySize());
- if (!C)
- return std::nullopt;
- assert(!Size.isScalable() && "Array elements cannot have a scalable size");
- Size *= C->getZExtValue();
- }
- return Size;
+ if (!isArrayAllocation())
+ return Size;
+ auto *C = dyn_cast<ConstantInt>(getArraySize());
+ if (!C)
+ return std::nullopt;
+ assert(!Size.isScalable() && "Array elements cannot have a scalable size");
+ auto checkedProd =
----------------
nikic wrote:
```suggestion
auto CheckedProd =
```
https://github.com/llvm/llvm-project/pull/97170
More information about the llvm-commits
mailing list