[flang-commits] [PATCH] D122467: [flang][lowering] Handle zero extent case in LBOUND

[Valentin Clement via Phabricator via flang-commits]

clementval accepted this revision.
clementval added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: flang/lib/Lower/ConvertVariable.cpp:1093
           loc, idxTy, genScalarValue(converter, loc, expr, symMap, stmtCtx));
       if (lowerBounds.empty())
+        result.emplace_back(Fortran::lower::genMaxWithZero(builder, loc, ub));
----------------
You should use brace on the if block if braces are used on the else block. 



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122467/new/

https://reviews.llvm.org/D122467