[PATCH] D25563: [ELF/AMDGPU]: Add support for new relocations

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 14:39:00 PDT 2016


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.



================
Comment at: ELF/Target.cpp:1489
+  case R_AMDGPU_REL32_HI:
+    write32le(Loc, (Val >> 32) & 0xFFFFFFFF);
+    break;
----------------
This is the same as

  write32le(Loc, Val >> 32);


https://reviews.llvm.org/D25563





More information about the llvm-commits mailing list