[PATCH] D119698: [flang] Lower basic function with scalar integer/logical return value

[Valentin Clement via Phabricator via llvm-commits]

clementval added inline comments.


================
Comment at: flang/lib/Lower/ConvertType.cpp:220-221
+        int kind = toInt64(Fortran::common::Clone(tySpec->kind())).value();
+        // llvm::SmallVector<Fortran::lower::LenParameterTy> params;
+        // translateLenParameters(params, tySpec->category(), ultimate);
+        ty = genFIRType(context, tySpec->category(), kind);
----------------
PeteSteinfeld wrote:
> Are these lines supposed to be here?
Nope. They will be part of a future patch. I'll remove them thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119698