@@ -607,7 +607,7 @@ lowerCirAttrAsValue(mlir::Operation *parentOp, cir::GlobalViewAttr globalAttr,
607
607
auto resTy = addrOp.getType ();
608
608
auto eltTy = converter->convertType (sourceType);
609
609
addrOp = rewriter.create <mlir::LLVM::GEPOp>(loc, resTy, eltTy, addrOp,
610
- indices, /* inbounds */ true );
610
+ indices, true );
611
611
}
612
612
613
613
auto ptrTy = mlir::dyn_cast<cir::PointerType>(globalAttr.getType ());
@@ -853,7 +853,7 @@ mlir::LogicalResult CIRToLLVMPtrStrideOpLowering::matchAndRewrite(
853
853
854
854
rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(ptrStrideOp, resultTy,
855
855
elementTy, adaptor.getBase (),
856
- index , /* inbounds*/ true );
856
+ index , /* inbounds= */ true );
857
857
return mlir::success ();
858
858
}
859
859
@@ -874,9 +874,8 @@ mlir::LogicalResult CIRToLLVMBaseClassAddrOpLowering::matchAndRewrite(
874
874
}
875
875
876
876
if (baseClassOp.getAssumeNotNull ()) {
877
- rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(baseClassOp, resultType,
878
- byteType, derivedAddr,
879
- offset, /* inbounds*/ true );
877
+ rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
878
+ baseClassOp, resultType, byteType, derivedAddr, offset);
880
879
} else {
881
880
auto loc = baseClassOp.getLoc ();
882
881
mlir::Value isNull = rewriter.create <mlir::LLVM::ICmpOp>(
@@ -902,8 +901,7 @@ mlir::LogicalResult CIRToLLVMDerivedClassAddrOpLowering::matchAndRewrite(
902
901
mlir::IntegerType::Signless);
903
902
if (derivedClassOp.getAssumeNotNull ()) {
904
903
rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(derivedClassOp, resultType,
905
- byteType, baseAddr, offset,
906
- /* inbounds*/ true );
904
+ byteType, baseAddr, offset);
907
905
} else {
908
906
auto loc = derivedClassOp.getLoc ();
909
907
mlir::Value isNull = rewriter.create <mlir::LLVM::ICmpOp>(
@@ -977,8 +975,8 @@ mlir::LogicalResult CIRToLLVMVTTAddrPointOpLowering::matchAndRewrite(
977
975
offsets.push_back (0 );
978
976
offsets.push_back (adaptor.getOffset ());
979
977
}
980
- rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
981
- op, resultType, eltType, llvmAddr, offsets, /* inbounds */ true );
978
+ rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(op, resultType, eltType,
979
+ llvmAddr, offsets, true );
982
980
return mlir::success ();
983
981
}
984
982
@@ -1035,7 +1033,7 @@ mlir::LogicalResult CIRToLLVMCastOpLowering::matchAndRewrite(
1035
1033
auto elementTy = convertTy (ptrTy.getPointee ());
1036
1034
auto offset = llvm::SmallVector<mlir::LLVM::GEPArg>{0 };
1037
1035
rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
1038
- castOp, targetType, elementTy, sourceValue, offset, /* inbounds */ true );
1036
+ castOp, targetType, elementTy, sourceValue, offset);
1039
1037
break ;
1040
1038
}
1041
1039
case cir::CastKind::int_to_bool: {
@@ -3239,8 +3237,8 @@ mlir::LogicalResult CIRToLLVMGetMemberOpLowering::matchAndRewrite(
3239
3237
// is always zero. The second offset tell us which member it will access.
3240
3238
llvm::SmallVector<mlir::LLVM::GEPArg, 2 > offset{0 , op.getIndex ()};
3241
3239
const auto elementTy = getTypeConverter ()->convertType (structTy);
3242
- rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
3243
- op, llResTy, elementTy, adaptor.getAddr (), offset, /* inbounds */ true );
3240
+ rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(op, llResTy, elementTy,
3241
+ adaptor.getAddr (), offset);
3244
3242
return mlir::success ();
3245
3243
}
3246
3244
case cir::StructType::Union:
@@ -3340,8 +3338,8 @@ mlir::LogicalResult CIRToLLVMVTableAddrPointOpLowering::matchAndRewrite(
3340
3338
}
3341
3339
3342
3340
assert (eltType && " Shouldn't ever be missing an eltType here" );
3343
- rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
3344
- op, targetType, eltType, symAddr, offsets, /* inbounds */ true );
3341
+ rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(op, targetType, eltType,
3342
+ symAddr, offsets, true );
3345
3343
3346
3344
return mlir::success ();
3347
3345
}
@@ -3840,8 +3838,8 @@ mlir::LogicalResult CIRToLLVMPtrMaskOpLowering::matchAndRewrite(
3840
3838
mlir::Value diff = rewriter.create <mlir::LLVM::SubOp>(loc, intPtr, masked);
3841
3839
rewriter.replaceOpWithNewOp <mlir::LLVM::GEPOp>(
3842
3840
op, getTypeConverter ()->convertType (op.getType ()),
3843
- mlir::IntegerType::get (moduleOp->getContext (), 8 ), adaptor.getPtr (), diff,
3844
- /* inbounds */ true );
3841
+ mlir::IntegerType::get (moduleOp->getContext (), 8 ), adaptor.getPtr (),
3842
+ diff );
3845
3843
return mlir::success ();
3846
3844
}
3847
3845
0 commit comments