[PATCH] D14957: [ELF/AArch64] Fix overflow checks for R_AARCH64_{ABS, PREL}{16, 32} relocations.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 25 10:29:32 PST 2015


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

LGTM


================
Comment at: ELF/Target.cpp:60-61
@@ +59,4 @@
+template <unsigned N> static void checkIntUInt(uint64_t V, uint32_t Type) {
+  if (isInt<N>(V) || isUInt<N>(V))
+    return;
+  StringRef S = getELFRelocationTypeName(Config->EMachine, Type);
----------------
Ah, you are right.


http://reviews.llvm.org/D14957





More information about the llvm-commits mailing list