================ @@ -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:
We gain better readability in this already complex file. I am not fully on board with memory footprint argument since I don't think a module will have that many instances of that struct that need to be created. Not a blocker from my side though since other reviewers are fine with it. https://github.com/llvm/llvm-project/pull/124746 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits