[PATCH] D29392: [ELF] - Change value checks to checkUInt for R_386_8/R_386_PC8/R_386_16/R_386_PC16 relocations.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 07:45:37 PST 2017

>So, with the fix for reading the addend as a signed number for the pc
>relocation, can you use checkInt for PC and checkUint for non-pc in

checkInts are already used for PC ones. And my new tests passes fine.
So yes, probably we can change non-pc to checkUint safely, since
that is more relaxed check in compare with checkInt.


More information about the llvm-commits mailing list