[PATCH] D83922: [OpenMP] Fix map clause for unused var: don't ignore it

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 16 11:40:35 PDT 2020


ABataev added a comment.

In D83922#2156510 <https://reviews.llvm.org/D83922#2156510>, @jdenny wrote:

> In D83922#2155749 <https://reviews.llvm.org/D83922#2155749>, @ABataev wrote:
>
> > I would add checks for mapping of `declare target to/link` vars and checked if they work in runtime.
>
>
> There are existing codegen tests for that, and they don't seem to be affected by this patch.  This patch only addresses the case where a `map` clause is specified for an unused variable.  Is there another behavior this patch might impact?


What I mean is the explicit mapping for `declare target to/link` variables. The variable is marked as declare to and then mapped. Do we have the tests for something like this?


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

https://reviews.llvm.org/D83922





More information about the cfe-commits mailing list