================
@@ -909,8 +917,9 @@ class MapInfoFinalizationPass
         builder.loadIfRef(op->getLoc(), baseAddr.getVarPtrPtr());
     mlir::omp::MapInfoOp newBaseAddrMapOp = mlir::omp::MapInfoOp::create(
         builder, op->getLoc(), loadBaseAddr.getType(), loadBaseAddr,
-        baseAddr.getVarTypeAttr(), baseAddr.getMapTypeAttr(),
-        baseAddr.getMapCaptureTypeAttr(), mlir::Value{}, members, membersAttr,
+        baseAddr.getVarPtrPtrTypeAttr(), baseAddr.getMapTypeAttr(),
----------------
bhandarkar-pranav wrote:

Should this be `builder.getVarPtrTypeAttr()`? But, at this point, i think a top 
level descriptor is being removed. So, if the descriptor hierarchy is being 
flattened, i think the pointee becomes the main type. So this might even be 
intentional. In either case a comment would be in good order here.

https://github.com/llvm/llvm-project/pull/177302
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to