[flang-commits] [PATCH] D113560: [fir] Add !fir.char type conversion

[Andrzej Warzynski via Phabricator via flang-commits]

awarzynski added inline comments.


================
Comment at: flang/lib/Optimizer/CodeGen/TypeConverter.h:159
+
+  // fir.char<n>  -->  llvm<"ix*">   where ix is scaled by kind mapping
+  mlir::Type convertCharType(fir::CharacterType charTy) {
----------------
[nit] Shouldn't this be:
* `!fir.char<n, 12>` --> `!llvm.array<12 x ix*>`
* `!fir.char<n, ?>` --> `ix*`
instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113560