jdenny added a comment. 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? 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