[PATCH] Initial ELF/AArch64 Support

Daniel Stewart stewartd at codeaurora.org
Tue Aug 5 07:01:51 PDT 2014


Thanks for the comments. I'll try to add some tests and resubmit a modified patch.

REPOSITORY
  rL LLVM

================
Comment at: lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp:352
@@ +351,3 @@
+    break;
+  case R_AARCH64_ABS64:
+    relocR_AARCH64_ABS64(location, relocVAddress, targetVAddress, ref.addend());
----------------
Renato Golin wrote:
> If I got it right, the ones that have debug messages here are not the ones that have it up there. Since we're mostly just calling them in the switch, why not move them all into one place, ie. the functions up there?
You're right, the debug messages should all be moved up into the actual relocation functions. I'll move them all up there and ensure all relocation functions have a debug message.

http://reviews.llvm.org/D4778






More information about the llvm-commits mailing list