Skip to content

Commit 4a746e5

Browse files
authored
[NFC][flang] Remove unused converter parameter. (#96854)
Removes an unused parameter in 2 utils that generate bounds ops.
1 parent eaf816f commit 4a746e5

File tree

2 files changed

+6
-9
lines changed

2 files changed

+6
-9
lines changed

flang/lib/Lower/DirectivesCommon.h

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -718,7 +718,6 @@ gatherBoundsOrBoundValues(fir::FirOpBuilder &builder, mlir::Location loc,
718718
template <typename BoundsOp, typename BoundsType>
719719
llvm::SmallVector<mlir::Value>
720720
genBoundsOpsFromBox(fir::FirOpBuilder &builder, mlir::Location loc,
721-
Fortran::lower::AbstractConverter &converter,
722721
fir::ExtendedValue dataExv,
723722
Fortran::lower::AddrAndBoundsInfo &info) {
724723
llvm::SmallVector<mlir::Value> bounds;
@@ -778,7 +777,6 @@ genBoundsOpsFromBox(fir::FirOpBuilder &builder, mlir::Location loc,
778777
template <typename BoundsOp, typename BoundsType>
779778
llvm::SmallVector<mlir::Value>
780779
genBaseBoundsOps(fir::FirOpBuilder &builder, mlir::Location loc,
781-
Fortran::lower::AbstractConverter &converter,
782780
fir::ExtendedValue dataExv, bool isAssumedSize) {
783781
mlir::Type idxTy = builder.getIndexType();
784782
mlir::Type boundTy = builder.getType<BoundsType>();
@@ -1163,7 +1161,7 @@ AddrAndBoundsInfo gatherDataOperandAddrAndBounds(
11631161
info.rawInput = info.addr;
11641162
if (mlir::isa<fir::SequenceType>(fir::unwrapRefType(info.addr.getType())))
11651163
bounds = genBaseBoundsOps<BoundsOp, BoundsType>(builder, operandLocation,
1166-
converter, compExv,
1164+
compExv,
11671165
/*isAssumedSize=*/false);
11681166
asFortran << designator.AsFortran();
11691167

@@ -1189,7 +1187,7 @@ AddrAndBoundsInfo gatherDataOperandAddrAndBounds(
11891187
info.addr = boxAddrOp.getVal();
11901188
info.rawInput = info.addr;
11911189
bounds = genBoundsOpsFromBox<BoundsOp, BoundsType>(
1192-
builder, operandLocation, converter, compExv, info);
1190+
builder, operandLocation, compExv, info);
11931191
}
11941192
} else {
11951193
if (detail::getRef<evaluate::ArrayRef>(designator)) {
@@ -1206,13 +1204,13 @@ AddrAndBoundsInfo gatherDataOperandAddrAndBounds(
12061204
if (mlir::isa<fir::BaseBoxType>(
12071205
fir::unwrapRefType(info.addr.getType()))) {
12081206
bounds = genBoundsOpsFromBox<BoundsOp, BoundsType>(
1209-
builder, operandLocation, converter, dataExv, info);
1207+
builder, operandLocation, dataExv, info);
12101208
}
12111209
bool dataExvIsAssumedSize =
12121210
Fortran::semantics::IsAssumedSizeArray(symRef->get().GetUltimate());
12131211
if (mlir::isa<fir::SequenceType>(fir::unwrapRefType(info.addr.getType())))
12141212
bounds = genBaseBoundsOps<BoundsOp, BoundsType>(
1215-
builder, operandLocation, converter, dataExv, dataExvIsAssumedSize);
1213+
builder, operandLocation, dataExv, dataExvIsAssumedSize);
12161214
asFortran << symRef->get().name().ToString();
12171215
} else { // Unsupported
12181216
llvm::report_fatal_error("Unsupported type of OpenACC operand");

flang/lib/Lower/OpenMP/OpenMP.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1738,15 +1738,14 @@ genTargetOp(lower::AbstractConverter &converter, lower::SymMap &symTable,
17381738
fir::unwrapRefType(info.addr.getType())))
17391739
bounds = lower::genBoundsOpsFromBox<mlir::omp::MapBoundsOp,
17401740
mlir::omp::MapBoundsType>(
1741-
firOpBuilder, converter.getCurrentLocation(), converter, dataExv,
1742-
info);
1741+
firOpBuilder, converter.getCurrentLocation(), dataExv, info);
17431742
if (mlir::isa<fir::SequenceType>(
17441743
fir::unwrapRefType(info.addr.getType()))) {
17451744
bool dataExvIsAssumedSize =
17461745
semantics::IsAssumedSizeArray(sym.GetUltimate());
17471746
bounds = lower::genBaseBoundsOps<mlir::omp::MapBoundsOp,
17481747
mlir::omp::MapBoundsType>(
1749-
firOpBuilder, converter.getCurrentLocation(), converter, dataExv,
1748+
firOpBuilder, converter.getCurrentLocation(), dataExv,
17501749
dataExvIsAssumedSize);
17511750
}
17521751

0 commit comments

Comments
 (0)