[flang-commits] [PATCH] D123520: [flang] Fix UBOUND() folding for constant arrays

[Mike K via Phabricator via flang-commits]

FruitClover added inline comments.


================
Comment at: flang/lib/Evaluate/fold-integer.cpp:75-79
+  template <typename T> ConstantSubscripts Get(const Parentheses<T> &x) {
     // LBOUND for (x) is [1, ..., 1] cause of temp variable inside
     // parentheses (lower bound is omitted, the default value is 1).
     return ConstantSubscripts(x.Rank(), ConstantSubscript{1});
   }
----------------
Proper `UBOUD((x))` is done in https://reviews.llvm.org/D123944 fyi (Did not want to mix changes and simplify review a bit)


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

https://reviews.llvm.org/D123520