[RuntimeDyld] Remove relocation bounds check introduced in r208375 (MachO only).
authorLang Hames <lhames@gmail.com>
Fri, 23 May 2014 18:35:44 +0000 (18:35 +0000)
committerLang Hames <lhames@gmail.com>
Fri, 23 May 2014 18:35:44 +0000 (18:35 +0000)
commit8e30e4b9b7d6e82aa1f1def07cfba78fef6c1084
tree4351563683a16660be5a63af64dd7a2227f5bebd
parent5de00f3b56805c7e980f049ceb3f166f8c69cec0
[RuntimeDyld] Remove relocation bounds check introduced in r208375 (MachO only).

We do all of our address arithmetic in 64-bit, and operations involving
logically negative 32-bit offsets (actually represented as unsigned 64 bit ints)
often overflow into higher bits. The overflow check could be preserved by
casting to uint32 at the callsite for applyRelocationValue, but this would
eliminate the value of the check.

The right way to handle overflow in relocations is to make relocation processing
target specific, and compute the values for RelocationEntry objects in the
appropriate types (32-bit for 32-bit targets, 64-bit for 64-bit targets). This
is coming as part of the cleanup I'm working on.

This fixes another i386 regression test.

<rdar://problem/16889891>

llvm-svn: 209536
llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.h