[llvm-branch-commits] [clang] [llvm] [mlir] [MLIR][OpenMP] Add LLVM translation support for OpenMP UserDefinedMappers (PR #124746)

[Kareem Ergawy via llvm-branch-commits]

================
@@ -2709,13 +2709,23 @@ getRefPtrIfDeclareTarget(mlir::Value value,
 }
 
 namespace {
+// Append customMappers information to existing MapInfosTy
+struct MapInfosTy : llvm::OpenMPIRBuilder::MapInfosTy {
+  SmallVector<Operation *, 4> Mappers;
----------------
ergawy wrote:

I understand we cannot merge all of this into `llvm::OpenMPIRBuilder::MapInfosTy`. I am talking about the new struct `MapInfosTy` that sets in between `llvm::OpenMPIRBuilder::MapInfosTy` and `MapInfoData`. I think we can make this new struct and `MapInfoData` one struct and use it in this file. I gave it a try and should be fine.

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