[flang-commits] [PATCH] D109227: [flang] GET_COMMAND_ARGUMENT(LENGTH) runtime implementation

Peter Klausler via Phabricator via flang-commits flang-commits at lists.llvm.org
Mon Sep 6 12:13:58 PDT 2021


klausler added inline comments.


================
Comment at: flang/runtime/command.cpp:14
 namespace Fortran::runtime {
-CppTypeFor<TypeCategory::Integer, 4> RTNAME(ArgumentCount)() {
+using LengthType = CppTypeFor<TypeCategory::Integer, 4>;
+
----------------
I don't think that this is the right solution.  What if the LENGTH= argument to GET_COMMAND_ARGUMENT had been an INTEGER*8 variable?  That case should work.  It would be better to use a larger result type here.


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

https://reviews.llvm.org/D109227



More information about the flang-commits mailing list