[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 15 15:12:16 PST 2019


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





More information about the cfe-commits mailing list