[PATCH] D12874: [LLD/ELF2] Initial support for local symbols in symtab
Davide Italiano via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 15 14:28:55 PDT 2015
davide removed rL LLVM as the repository for this revision.
davide updated this revision to Diff 34835.
davide added a comment.
Hopefully addressed (almost) all the comments.
- Cleaned up tests
- Fixup crash reported by Rafael + added a new test for that
- Rename variables
I'll refactor getLocalSymbols() and getNonLocalSymbols() in a separate NFC patch. I put an XXX on top of the function so I won't forget.
Also, I think I ran clang-format correctly on my svn checkout, but please double check.
http://reviews.llvm.org/D12874
Files:
ELF/Config.h
ELF/Driver.cpp
ELF/InputFiles.cpp
ELF/InputFiles.h
ELF/Options.td
ELF/Writer.cpp
test/elf2/basic64be.s
test/elf2/local-dynamic.s
test/elf2/local.s
test/elf2/resolution.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12874.34835.patch
Type: text/x-patch
Size: 12425 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150915/39932a97/attachment.bin>
More information about the llvm-commits
mailing list