OSDN Git Service

[PowerPC] Correctly handle fixups of other than 4 byte size
authorUlrich Weigand <ulrich.weigand@de.ibm.com>
Wed, 15 May 2013 15:01:46 +0000 (15:01 +0000)
committerUlrich Weigand <ulrich.weigand@de.ibm.com>
Wed, 15 May 2013 15:01:46 +0000 (15:01 +0000)
commitb1cf8de85af10ff3d4ded6431be1cebd6133fd54
tree8cb9d6d90916bd05f0b0ea3db55fd6287fe148de
parent38286bee4d9195c622fee59e485fcb8c18fd5b3b
[PowerPC] Correctly handle fixups of other than 4 byte size

The PPCAsmBackend::applyFixup routine handles the case where a
fixup can be resolved within the same object file.  However,
this routine is currently hard-coded to assume the size of
any fixup is always exactly 4 bytes.

This is sort-of correct for fixups on instruction text; even
though it only works because several of what really would be
2-byte fixups are presented as 4-byte fixups instead (requiring
another hack in PPCELFObjectWriter::adjustFixupOffset to clean
it up).

However, this assumption breaks down completely for fixups
on data, which legitimately can be of any size (1, 2, 4, or 8).

This patch makes applyFixup aware of fixups of varying sizes,
introducing a new helper routine getFixupKindNumBytes (along
the lines of what the ARM back end does).  Note that in order
to handle fixups of size 8, we also need to fix the return type
of adjustFixupValue to uint64_t to avoid truncation.

Tested on both 64-bit and 32-bit PowerPC, using external and
integrated assembler.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@181891 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/PowerPC/MCTargetDesc/PPCAsmBackend.cpp