[PATCH] D41057: Avoid a gnu readelf warning
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 11 10:51:59 PST 2017
ruiu added inline comments.
================
Comment at: ELF/SyntheticSections.cpp:1233
+ if (this == InX::RelaPlt) {
+ Info = InX::GotPlt->getParent()->SectionIndex;
+ getParent()->Info = Info;
----------------
grimar wrote:
> emaste wrote:
> > Why `.got.plt` and not `.plt`?
> Do we really want to care about readelf warnings about something that is not in ELF spec (I assume it is not) ? While it is not a error... may be we can just ignore it ?
I think I feel the same as George. Is this an issue?
Can you always set or always not set `Info`?
https://reviews.llvm.org/D41057
More information about the llvm-commits
mailing list