[PATCH] D89994: [libomptarget][nvptx] Undef, internal shared variables

[Alexey Bataev via Phabricator via cfe-commits]

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