[llvm-commits] [patch] Allow >256 ELF relocations

Tim Northover Tim.Northover at arm.com
Mon Jan 7 06:58:38 PST 2013


Hi all,

The ELF64 specification says that 32 bits of the r_info field are used for the
relocation type; however the ELF support code currently uses "unsigned char"
to store this type. Current targets work because they have <= 256 relocations,
but AArch64 defines more (or, at least, spreads them out more).

This patch switches the types used so that everything works properly.

It's part of the change to generic LLVM code that we had to implement while
developing the AArch64 backend. These are the changes that no other target
exercises, so sending them to the list before would have been a little
motive-less, and often untestable.

This one is reasonably simple, OK to commit?

Cheers.

Tim.

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: elf-reloc-size.diff
Type: text/x-patch
Size: 1762 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130107/bc97c9d8/attachment.bin>


More information about the llvm-commits mailing list