[llvm-branch-commits] [flang] [llvm] [mlir] [OpenMP][MLIR] Modify OpenMP Dialect lowering to support attach mapping (PR #179023)

[Sergio Afonso via llvm-branch-commits]

================
@@ -5303,6 +5348,121 @@ static bool checkIfPointerMap(omp::MapInfoOp mapOp) {
   return false;
 }
 
+// This function handles the insertion of a single item of map data from
+// MapInfoData into the OMPIRBuilders MapInfo list. Utilising this function
+// means the map being inserted can be treated as a non-parent map entity,
+// if the memberOfFlag is set then the map being inserted is treated as
+// a member map of a larger entity. The insertion into the MapInfo list of
+// the OMPIRBuilder can vary based on a number of factors, such as if it's
+// a ref_ptr or ref_ptee map, if it's a member of a record, what construct
+// the map belongs to and the various map type bit flags that are set for
+// the map.
----------------
skatrak wrote:

```suggestion
/// This function handles the insertion of a single item of map data from
/// MapInfoData into the OMPIRBuilder's MapInfo list. Utilising this function
/// means the map being inserted can be treated as a non-parent map entity.
/// If the memberOfFlag is set, then the map being inserted is treated as
/// a member map of a larger entity. The insertion into the MapInfo list of
/// the OMPIRBuilder can vary based on a number of factors, such as if it's
/// a ref_ptr or ref_ptee map, if it's a member of a record, what construct
/// the map belongs to and the various map type bit flags that are set for
/// the map.
```

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