[llvm-branch-commits] [flang] [MLIR][OpenMP] Add Lowering support for OpenMP custom mappers in map clause (PR #121001)

[Kareem Ergawy via llvm-branch-commits]

================
@@ -1057,13 +1075,17 @@ bool ClauseProcessor::processMap(
            "Support for iterator modifiers is not implemented yet");
     }
     if (mappers) {
-      TODO(currentLocation,
-           "Support for mapper modifiers is not implemented yet");
+      assert(mappers->size() == 1 && "more than one mapper");
+      mapperIdName = mappers->front().v.id().symbol->name().ToString();
+      if (mapperIdName != "default")
----------------
ergawy wrote:

I think it would be better to do this processing of the `mapperIdName` inside `processMapObjects` since we do further processing of the name in that function anyway ([line 975](https://github.com/llvm/llvm-project/pull/121001/files#diff-55798c5090a8f8499f773b3fb46fb98a7aabe0f58ec60ff295e107acb36c7707R975) in `ClauseProcessor.cpp`).

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