Skip to content

Commit 502ccd8

Browse files
authored
[MLIR][OpenMP] NFC: Remove redundant check (#91594)
The check removed by this patch in the OpenMP to LLVM IR translation pass already exists as part of the op verifier for `omp.loop_nest`.
1 parent a43d2f6 commit 502ccd8

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -939,10 +939,6 @@ convertOmpWsloop(Operation &opInst, llvm::IRBuilderBase &builder,
939939
llvm::ArrayRef<bool> isByRef = getIsByRef(wsloopOp.getReductionVarsByref());
940940
assert(isByRef.size() == wsloopOp.getNumReductionVars());
941941

942-
// TODO: this should be in the op verifier instead.
943-
if (loopOp.getLowerBound().empty())
944-
return failure();
945-
946942
// Static is the default.
947943
auto schedule =
948944
wsloopOp.getScheduleVal().value_or(omp::ClauseScheduleKind::Static);

0 commit comments

Comments
 (0)