[PATCH] D40962: [ELF] When a relocation is out of range print the value and the range

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 01:40:07 PST 2017


grimar added a comment.

Patch LGTM, please wait for Rui approval.



================
Comment at: ELF/Target.h:150
+  error(getErrorLocation(Loc) + "relocation " + lld::toString(Type) +
+        " out of range: requested value " + V + " is not in [" + Twine(Min) +
+        ", " + Twine(Max) + "]");
----------------
nit: I would shorten this:
relocation R_386_PC16 out of range: 65536 is not in [-65536, 65535]
It is the matter of taste though.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D40962





More information about the llvm-commits mailing list