From dd50e26f8a844aa7dc9908b94bce66188c4d4e91 Mon Sep 17 00:00:00 2001 From: Alfonso Gregory Date: Tue, 21 Sep 2021 11:15:00 -0400 Subject: [PATCH] [NFC][IR] Remove unused assignment to Offset This value is overwritten anyway, so let's remove it --- llvm/lib/IR/Operator.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/llvm/lib/IR/Operator.cpp b/llvm/lib/IR/Operator.cpp index 0e26311..4e400ad 100644 --- a/llvm/lib/IR/Operator.cpp +++ b/llvm/lib/IR/Operator.cpp @@ -63,7 +63,7 @@ Align GEPOperator::getMaxPreservedAlignment(const DataLayout &DL) const { Align Result = Align(llvm::Value::MaximumAlignment); for (gep_type_iterator GTI = gep_type_begin(this), GTE = gep_type_end(this); GTI != GTE; ++GTI) { - int64_t Offset = 1; + uint64_t Offset; ConstantInt *OpC = dyn_cast(GTI.getOperand()); if (StructType *STy = GTI.getStructTypeOrNull()) { @@ -71,11 +71,9 @@ Align GEPOperator::getMaxPreservedAlignment(const DataLayout &DL) const { Offset = SL->getElementOffset(OpC->getZExtValue()); } else { assert(GTI.isSequential() && "should be sequencial"); - /// If the index isn't know we take 1 because it is the index that will + /// If the index isn't known, we take 1 because it is the index that will /// give the worse alignment of the offset. - int64_t ElemCount = 1; - if (OpC) - ElemCount = OpC->getZExtValue(); + const uint64_t ElemCount = OpC ? OpC->getZExtValue() : 1; Offset = DL.getTypeAllocSize(GTI.getIndexedType()) * ElemCount; } Result = Align(MinAlign(Offset, Result.value())); -- 2.7.4