[llvm-branch-commits] [flang] [Flang][OpenMP] Derived type explicit allocatable member mapping (PR #96266)

[Kareem Ergawy via llvm-branch-commits]

================
@@ -52,12 +52,22 @@ using DeclareTargetCapturePair =
 struct OmpMapMemberIndicesData {
   // The indices representing the component members placement in its derived
   // type parents hierarchy.
-  llvm::SmallVector<int> memberPlacementIndices;
+  llvm::SmallVector<llvm::SmallVector<int>> memberPlacementIndices;
 
   // Placement of the member in the member vector.
-  mlir::omp::MapInfoOp memberMap;
+  llvm::SmallVector<mlir::omp::MapInfoOp> memberMap;
 };
 
+llvm::SmallVector<int>
+generateMemberPlacementIndices(const Object &object,
----------------
ergawy wrote:

There is already a version of `generateMemberPlacementIndices` that takes a `SmallVector` param as an output param. And both versions have exactly the same logic. Is there a reason for duplication?

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