ABataev added a comment.

In D89994#2348655 <https://reviews.llvm.org/D89994#2348655>, @JonChesterfield 
wrote:

> The nvptx back end accepts common + zero + shared, but not common + undef + 
> shared. I think weak_odr is conceptually right here, but given the warning 
> that nvlink doesn't support weak symbols, internal also seems fine. Can 
> someone see an advantage to weak over internal? It could be arch specific at 
> the risk of a lot of test duplication.

IIRC, it supports weak symbols, but does not support weak symbols of different 
sizes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89994

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to