[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