[llvm-commits] ELFReader.cpp update - Add support for References.

Sid Manning sidneym at codeaurora.org
Fri Sep 14 14:15:22 PDT 2012


Update to ReaderELF.cpp, ELF.h and test directory.

-- Adds a testcase to check relocations are emitted.  Looks like 
TestingHelpers will need to to be updated in the near future for the 
many relocation types elf will likely support.
-- Addresses style issues.
-- Adds doxygen comments to Atoms classes and elaborates on some routines.
-- Changes DefinedAtom's declaration of ELFReference to a reference.
-- Some loops changed to c++11 style.
-- findAtom uses DenseMap::lookup instead of looping.
-- Uses the iterator added to ELF.h to step through the SHT_REL/RELA 
sections.  Thanks, Michael Spencer for the iterator in ELF.h.

Thanks,


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
hosted by The Linux Foundation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ELF.h.diff
Type: text/x-patch
Size: 4162 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120914/0dba3301/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ReaderELF.cpp.diff
Type: text/x-patch
Size: 30711 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120914/0dba3301/attachment-0001.bin>


More information about the llvm-commits mailing list