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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
