[flang-commits] [PATCH] D111785: [flang] runtime: Read environment variables directly

Peter Klausler via Phabricator via flang-commits flang-commits at lists.llvm.org
Wed Oct 27 08:57:31 PDT 2021


klausler added inline comments.


================
Comment at: flang/runtime/environment.cpp:76
+    const char *name, std::size_t name_length, const Terminator &terminator) {
+  assert(name && name_length && "Invalid environment variable name");
 
----------------
Please use RUNTIME_CHECK instead of assert, here and below.


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

https://reviews.llvm.org/D111785



More information about the flang-commits mailing list