[llvm-branch-commits] [flang] [Flang][OpenMP] - When mapping a `fir.boxchar`, map the underlying data pointer as a member (PR #141715)

[via llvm-branch-commits]

================
@@ -285,6 +285,9 @@ bool hasDynamicSize(mlir::Type t) {
     return true;
   if (auto rec = mlir::dyn_cast<fir::RecordType>(t))
     return hasDynamicSize(rec);
+  if (auto boxChar = mlir::dyn_cast<fir::BoxCharType>(t)) {
+    return characterWithDynamicLen(boxChar.getEleTy());
+  }
----------------
jeanPerier wrote:

This `hasDynamicSize` change brings ambiguity to the helper because this helpers return false for pointer like types (fir.ref, fir.box, ..), it is meant to deal with data type. fir.boxchar is a pointer like type, not a data type like.

It is up to the user I think to call it with the data type (fir.char<....>).

https://github.com/llvm/llvm-project/pull/141715