Convert all common symbols to regular symbols after scan.
This means that the downstream code does not to handle common symbols as a special case.
Paths
| Differential D38137
[ELF] Simpler scheme for handling common symbols ClosedPublic Authored by bd1976llvm on Sep 21 2017, 8:36 AM.
Details Summary Convert all common symbols to regular symbols after scan. This means that the downstream code does not to handle common symbols as a special case.
Diff Detail
Event TimelineComment Actions First of all, could you read https://llvm.org/docs/CodingStandards.html and follow that style? One of the easiest way is to run clang-format-diff on your patch so that everything in your patch is formatted automatically.
Comment Actions
What a mess! Thanks for reviewing the patch despite how unstylish it is.
bd1976llvm marked 5 inline comments as done. Comment ActionsUpdated as per review comments.
bd1976llvm added inline comments.
This revision is now accepted and ready to land.Sep 28 2017, 1:08 PM Closed by commit rL314495: [ELF] Simpler scheme for handling common symbols (authored by bd1976llvm). · Explain WhySep 29 2017, 2:10 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 117101 lld/trunk/ELF/Driver.cpp
lld/trunk/ELF/MarkLive.cpp
lld/trunk/ELF/Symbols.h
lld/trunk/ELF/Symbols.cpp
lld/trunk/ELF/SyntheticSections.h
lld/trunk/ELF/SyntheticSections.cpp
lld/trunk/ELF/SyntheticSections.cpp~RF2791d30b.TMP
lld/trunk/ELF/Writer.cpp
|