[flang-commits] [flang] [flang][OpenMP][DoConcurrent] Emit declare mapper for records (PR #179936)

[Pranav Bhandarkar via flang-commits]

================
@@ -59,6 +60,14 @@ mlir::Value mapTemporaryValue(fir::FirOpBuilder &firOpBuilder,
 /// maps.
 void cloneOrMapRegionOutsiders(
     fir::FirOpBuilder &firOpBuilder, mlir::omp::TargetOp targetOp);
+
+using RecordMemberMapperMangler =
+    std::function<void(std::string &mapperId, llvm::StringRef memberName)>;
+
+mlir::FlatSymbolRefAttr getOrGenImplicitDefaultDeclareMapper(
----------------
bhandarkar-pranav wrote:

It looks like this has been moved as is to `Utils` so other optimization passes like `DoConcurrentConversion` can  use it. Even so, may be this is a good time to add a comment here about the parameters especially if there are any expectations around `recordType`, `mapperNameStr` and `mangler`

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