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

Reply via email to