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

[Jean Perier via Phabricator via flang-commits]

jeanPerier marked an inline comment as done.
jeanPerier added inline comments.


================
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));
----------------
clementval wrote:
> You should use brace on the if block if braces are used on the else block. 
> 
Thanks, the brace are actually useless. I have removed them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122467