[PATCH] D21654: ELF/AMDGPU: Add support for R_AMDGPU_ABS32

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 10:23:06 PDT 2016


ruiu added a comment.

The fact that ABS32 is generated only for DWARF does not necessarily mean that you need to create a DWARF to verify the behavior of the relocation. I was trying to say was that the test is too big compared to the size of the feature that is testing; it mostly verifies the structure of DWARF rather than the relocation. A test for ABS32 can be written in a few lines.


https://reviews.llvm.org/D21654





More information about the llvm-commits mailing list