[PATCH] D66247: [OpenMP] Fix target map for unused variables

Joel E. Denny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 14 14:58:33 PDT 2019


jdenny added a comment.

In D66247#1630352 <https://reviews.llvm.org/D66247#1630352>, @ABataev wrote:

> Yes, just realized that, defaultmap does not affect explicit firstprivates. Then just check `map(a) firstprivate(a)` for `int128` type. Check that it still has `tofrom` mapping. If so, then probably we already can handle such combinations correctly in the codegen. Also, test it for other mapping types, like alloc, from, to.


All work as expected.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66247





More information about the cfe-commits mailing list