Skip to content

Commit b21af72

Browse files
committed
Remove unused stand-alone tiling code. Fix typo.
1 parent f59c027 commit b21af72

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

flang/lib/Lower/OpenMP/OpenMP.cpp

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1601,22 +1601,12 @@ genLoopNestClauses(lower::AbstractConverter &converter,
16011601
clauseOps.loopInclusive = converter.getFirOpBuilder().getUnitAttr();
16021602

16031603
fir::FirOpBuilder &firOpBuilder = converter.getFirOpBuilder();
1604-
for (auto &clause : clauses) {
1604+
for (auto &clause : clauses)
16051605
if (clause.id == llvm::omp::Clause::OMPC_collapse) {
16061606
const auto &collapse = std::get<clause::Collapse>(clause.u);
16071607
int64_t collapseValue = evaluate::ToInt64(collapse.v).value();
16081608
clauseOps.numCollapse = firOpBuilder.getI64IntegerAttr(collapseValue);
1609-
} else if (clause.id == llvm::omp::Clause::OMPC_sizes) {
1610-
// This case handles the stand-alone tiling construct
1611-
const auto &sizes = std::get<clause::Sizes>(clause.u);
1612-
llvm::SmallVector<int64_t> sizeValues;
1613-
for (auto &size : sizes.v) {
1614-
int64_t sizeValue = evaluate::ToInt64(size).value();
1615-
sizeValues.push_back(sizeValue);
1616-
}
1617-
clauseOps.tileSizes = sizeValues;
16181609
}
1619-
}
16201610

16211611
llvm::SmallVector<int64_t> sizeValues;
16221612
auto *ompCons{eval.getIf<parser::OpenMPConstruct>()};
@@ -3895,7 +3885,7 @@ static void genOMP(lower::AbstractConverter &converter, lower::SymMap &symTable,
38953885
switch (nestedDirective) {
38963886
case llvm::omp::Directive::OMPD_tile:
38973887
// Skip OMPD_tile since the tile sizes will be retrieved when
3898-
// generating the omp.looop_nest op.
3888+
// generating the omp.loop_nest op.
38993889
break;
39003890
default: {
39013891
unsigned version = semaCtx.langOptions().OpenMPVersion;

0 commit comments

Comments
 (0)