[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