[llvm] [AMDGPU] Fixed llvm-debuginfo-analyzer for AMDGPU. (PR #145125)
Adam Yang via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 25 12:05:55 PDT 2025
adam-yang wrote:
> Thank you for the patch!
>
> I have some small nits, and one larger concern around changing `AMDPAL` relocation behavior. If I understand the intent of the change, I think that part of the change should be removed and instead support for the REL relocations added in LogicalView or whatever library it depends on
Thanks for the review @slinder1! This change was the precursor for #149429, which is intended to be a replacement for this change. Did you have a chance to take a look at that? I'm open to get this change in first, and reduce #149429 to a smaller PR if that makes things easier.
https://github.com/llvm/llvm-project/pull/145125
More information about the llvm-commits
mailing list