[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
>  values.




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.


http://reviews.llvm.org/D18972





More information about the llvm-commits mailing list