[flang-commits] [PATCH] D136151: [flang] Introduce FortranVariableOpInterface for ops creating variable

[Slava Zakharin via Phabricator via flang-commits]

vzakhari added inline comments.


================
Comment at: flang/include/flang/Optimizer/Dialect/FortranVariableInterface.td:111
+    bool hasExplicitCharLen() {
+      return isCharacter() && !getExplicitTypeParams().empty();
+    }
----------------
Is it worth asserting that `getExplicitTypeParams` returns an empty range or a range with exactly one element, when `isCharacter` is true?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136151