[PATCH] D113763: [flang][CodeGen] Transform `fir.boxchar_len` to a sequence of LLVM MLIR

[Valentin Clement via Phabricator via llvm-commits]

clementval accepted this revision.
clementval added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: flang/lib/Optimizer/CodeGen/CodeGen.cpp:1464
+    mlir::LLVM::ExtractValueOp len = genExtractValueWithIndex(
+        loc, boxChar, boxChar.getType(), rewriter, ctx, 1);
+    mlir::Value lenAfterCast = integerCast(loc, rewriter, returnValTy, len);
----------------
nit: maybe having a constexpr with a proper name could help here instead of 1 or maybe a small comment. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113763