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

[Akash Banerjee 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")
----------------
TIFitis wrote:

We would then have to pass the `mappers` element from the parser struct to the `processMapObjects` function. I guess it's better to just pass the string.

Also, in a future patch when adding support for implicit default mapping, I expect to consolidate all the process here, and remove the code inside `processMapObjects`.

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