Skip to content

Commit 69d068d

Browse files
address minor review comment
1 parent 92abc26 commit 69d068d

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

flang/lib/Lower/OpenMP/OpenMP.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -785,6 +785,7 @@ genEnterExitUpdateDataOp(Fortran::lower::AbstractConverter &converter,
785785
ClauseProcessor cp(converter, semaCtx, clauseList);
786786
cp.processIf(directiveName, ifClauseOperand);
787787
cp.processDevice(stmtCtx, deviceOperand);
788+
cp.processDepend(dependTypeOperands, dependOperands);
788789
cp.processNowait(nowaitAttr);
789790

790791
if constexpr (std::is_same_v<OpTy, mlir::omp::UpdateDataOp>) {
@@ -796,7 +797,6 @@ genEnterExitUpdateDataOp(Fortran::lower::AbstractConverter &converter,
796797
} else {
797798
cp.processMap(currentLocation, directive, stmtCtx, mapOperands);
798799
}
799-
cp.processDepend(dependTypeOperands, dependOperands);
800800

801801
return firOpBuilder.create<OpTy>(
802802
currentLocation, ifClauseOperand, deviceOperand,
@@ -982,10 +982,11 @@ genTargetOp(Fortran::lower::AbstractConverter &converter,
982982
ifClauseOperand);
983983
cp.processDevice(stmtCtx, deviceOperand);
984984
cp.processThreadLimit(stmtCtx, threadLimitOperand);
985+
cp.processDepend(dependTypeOperands, dependOperands);
985986
cp.processNowait(nowaitAttr);
986987
cp.processMap(currentLocation, directive, stmtCtx, mapOperands, &mapSymTypes,
987988
&mapSymLocs, &mapSymbols);
988-
cp.processDepend(dependTypeOperands, dependOperands);
989+
989990
cp.processTODO<Fortran::parser::OmpClause::Private,
990991
Fortran::parser::OmpClause::Firstprivate,
991992
Fortran::parser::OmpClause::IsDevicePtr,

0 commit comments

Comments
 (0)