ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land.
In D58074#1399881 <https://reviews.llvm.org/D58074#1399881>, @lildmh wrote: > Thanks for the catch! I also change the name from `OMPMappableExprListLocTy` > to `OMPVarListLocTy` to be more accurate. We can potentially factorize other > varlist clause code with this. But it will be too large for this patch. Thanks for fixing this. Yes, if you can think we improve something in the code, please, go ahead in the next patches. LGTM! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58074/new/ https://reviews.llvm.org/D58074 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
