[PATCH] D58523: [OpenMP 5.0] Parsing/sema support for to clause with mapper modifier

[Lingda Li via Phabricator via cfe-commits]

lildmh added inline comments.


================
Comment at: lib/Sema/SemaOpenMP.cpp:13195
-      } else {
-        MVLI.UDMapperList.push_back(nullptr);
       }
----------------
ABataev wrote:
> Is this correct for `from` clause?
Yes, it's correct for `from`, which will never use `MVLI.UDMapperList`. 
Regression test is also correct.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58523/new/

https://reviews.llvm.org/D58523



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits