[PATCH] D25693: [AMDGPU] Emit constant address space data in .rodata section and use relocations instead of fixups (amdhsa only)
Tom Stellard via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 19 11:53:26 PDT 2016
tstellarAMD added inline comments.
================
Comment at: test/CodeGen/AMDGPU/global-constant.ll:4-6
@readonly = private unnamed_addr addrspace(2) constant [4 x float] [float 0.0, float 1.0, float 2.0, float 3.0]
@readonly2 = private unnamed_addr addrspace(2) constant [4 x float] [float 4.0, float 5.0, float 6.0, float 7.0]
----------------
I think this patch also fixes the case where we have an external variable in address space 2. I would add a test for that here. If I'm wrong and this patch doesn't fix this, then we can add the test case along with the fix for it in a follow-up patch.
https://reviews.llvm.org/D25693
More information about the llvm-commits
mailing list