[PATCH] D15766: Delete DefinedAbsolute

[Rafael Ávila de Espíndola via llvm-commits]

rafael created this revision.
rafael added reviewers: ruiu, grimar.
rafael added a subscriber: llvm-commits.

There are 3 symbol types that a .bc can provide during lto: defined, undefined, common.

Defined and undefined symbols have already been refactored. I was working on common and noticed that absolute symbols would become an oddity: They would be the only symbol type present in a .o but not in a.bc.

Looking a bit more, other than the special section number they were only used for special rules for computing values. In that way they are similar to TLS, and we don't have a DefinedTLS.

This patch deletes it. With it we have a reasonable rule of the thumb for having a symbol kind: It exists if it has special resolution semantics.


http://reviews.llvm.org/D15766

Files:
  ELF/Driver.cpp
  ELF/InputFiles.cpp
  ELF/InputSection.cpp
  ELF/MarkLive.cpp
  ELF/OutputSections.cpp
  ELF/SymbolTable.cpp
  ELF/Symbols.cpp
  ELF/Symbols.h
  ELF/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15766.43580.patch
Type: text/x-patch
Size: 12759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151224/92d82aae/attachment.bin>