[PATCH] D18349: [ELF][MIPS] Reduce number of redundant entries in the local part of MIPS GOT
Simon Atanasyan via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 29 02:55:35 PDT 2016
atanasyan updated the summary for this revision.
atanasyan updated this revision to Diff 51883.
atanasyan added a comment.
- Fixed the comment describes how we count the number of required GOT entries.
- Removed changes from Writer.cpp. I reviewed this code once again and realized that the changes can be rolled back. The only section (except .dynamic, .got.plt etc) which can modify its sh_size in the finalize method is the MergeOutputSection. But such sections always go before the .got section in the OutputSections container because we support only read-only merge sections and read-only sections go before writable ones. So it is does not necessary to postpone finalizing of the .got section. It would be nice to have an assert to check that fact but I cannot find how to do that without significant modification of the code.
Repository:
rL LLVM
http://reviews.llvm.org/D18349
Files:
ELF/OutputSections.cpp
ELF/OutputSections.h
test/ELF/mips-got-redundant.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18349.51883.patch
Type: text/x-patch
Size: 4728 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160329/d4243996/attachment.bin>
More information about the llvm-commits
mailing list