[PATCH] D29214: [AMDGPU] Internalize non-kernel symbols

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 18:37:57 PST 2017


rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: test/CodeGen/AMDGPU/internalize.ll:7-8
+
+ at gvar_unused = addrspace(1) global i32 undef, align 4
+ at gvar_used = addrspace(1) global i32 undef, align 4
+
----------------
arsenm wrote:
> Should this check with various linkage types that should remain visible?
As long as there is no late linking there are no linkages which should remain visible. Do you have an example?


Repository:
  rL LLVM

https://reviews.llvm.org/D29214





More information about the llvm-commits mailing list