[PATCH] D76440: [AMDGPU] Add Relocation Constant Support

Yong He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 20 11:55:31 PDT 2020


csyonghe added a comment.

This is suggested by @nhaehnle 
I like this idea because it does not lock into any specific data structure prematurely in amdgcn, which the backend really does not care. In the end you want to control the name of the symbol exported in the elf.
This will enable us to use the intrinsic for many different purposes in llpc, and push the responsibility of naming a relocation symbol to llpc instead of in amdgpu.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76440/new/

https://reviews.llvm.org/D76440





More information about the llvm-commits mailing list