[PATCH] D18972: [AMDGPU] Add basic relocation support (currently used for dwarf only)
Tom Stellard via llvm-commits
llvm-commits at lists.llvm.org
Fri May 27 10:10:45 PDT 2016
tstellarAMD added a comment.
In http://reviews.llvm.org/D18972#442403, @rafael wrote:
> If this is just for dwarf, why do you need high and lo relocations?
> On x86_64 we use 32 and 64 relocations, but they apply to 32 and 64 bit
In http://reviews.llvm.org/D18972#442405, @rafael wrote:
> In fact, you have no .s that produces a high relocation.
This patch adds all the relocation definitions, but only uses the ones required for dwarf. I'm working on patches to add support for relocations of global variables, which use the high relocations.
More information about the llvm-commits