[PATCH] D13558: [ELF2] Make VAStart a Target-dependent property

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 8 09:57:50 PDT 2015


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.h:42
@@ -40,2 +41,3 @@
 protected:
+  unsigned VAStart;
   unsigned PCRelReloc;
----------------
I think this has to be uint64_t. Otherwise it can't handle large addresses when cross-linking 32-bit host/64-bit target.


http://reviews.llvm.org/D13558





More information about the llvm-commits mailing list