agozillon wrote:

> Thank you for the PR @agozillon. I have a few comments inline in the PR. One 
> more thing - I do not see the helper function `createMapInfoOp` in 
> flang/lib/Utils/OpenMP.cpp updated for the new `varPtrPtrType` in this PR.

Thank you very much for the quick review @bhandarkar-pranav and sorry for the 
slow reply/update. Will hopefully get to updating the stack early next week. 
But working through the feedback just now slowly :-)

I believe it is updated in this PR, I think this is what you mean (do correct 
me if I'm wrong though please): 
https://github.com/llvm/llvm-project/pull/177302/changes#diff-729c24525777c2d3ddd5012a08e5c81ad925ce69300cd3bb0a069634aa7eab8fR29
 

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