[flang-commits] [PATCH] D102857: [flang] Implement more transformational intrinsic functions in runtime

[Pete Steinfeld via Phabricator via flang-commits]

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

All builds, tests, and looks good.



================
Comment at: flang/runtime/transformational.cpp:13-17
+// Many of these are defined in the 2018 standard with text that makes sense
+// only if argument arrays have lower bounds of one.  Rather than interpret
+// these cases as implying a hidden constraint, these implementations
+// work with arbitrary lower bounds.  This may be technically an extension
+// of the standard but it more likely to conform with its intent.
----------------
Should we ask Mark to attempt to clarify this with the on-going standards work?


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

https://reviews.llvm.org/D102857