[flang-commits] [PATCH] D123157: [flang] Add runtime API to catch unit number out of range

[Peter Klausler via Phabricator via flang-commits]

klausler accepted this revision.
klausler added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/include/flang/Runtime/iostat.h:70
   IostatUTF8Decoding,
+  IostatUnitOverflow,
 };
----------------
IostatErrorString() in iostat.cpp will need a `case` for this new enumeration (for completeness).


================
Comment at: flang/runtime/io-api.cpp:1299
+      errorHandler.SignalError(
+          IostatUnitOverflow, "UNIT number is out of range");
+    }
----------------
This string argument can be omitted once you add it to IostatErrorString in iostat.cpp.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123157