[PATCH] D84352: [flang] Fix an assert when RESHAPE() is called on empty strings

[Peter Klausler via Phabricator via llvm-commits]

klausler added inline comments.


================
Comment at: flang/lib/Evaluate/constant.cpp:257
+    while (copied < count) {
+      if (copied < values_.size()) {
+        auto *dest{&values_.at(SubscriptsToOffset(resultSubscripts) * length_)};
----------------
This new 'if' statement might just hide bugs.  Can `copied` be >= `values_.size()` and still be valid?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84352