[lld] r299615 - Create GOT and PLT entries early in scanRelocs().
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 6 15:29:58 PDT 2017
On Thu, Apr 6, 2017 at 1:11 PM, Rafael EspĂndola <rafael.espindola at gmail.com
> wrote:
> > static bool refersToGotEntry(RelExpr Expr) {
> > return isRelExprOneOf<R_GOT, R_GOT_OFF, R_MIPS_GOT_LOCAL_PAGE,
> R_MIPS_GOT_OFF,
> > - R_MIPS_GOT_OFF32, R_MIPS_TLSGD, R_MIPS_TLSLD,
> > - R_GOT_PAGE_PC, R_GOT_PC, R_GOT_FROM_END,
> R_TLSGD,
> > - R_TLSGD_PC, R_TLSDESC, R_TLSDESC_PAGE>(Expr);
> > + R_MIPS_GOT_OFF32, R_GOT_PAGE_PC, R_GOT_PC,
> > + R_GOT_FROM_END>(Expr);
> > }
>
> R_TLSGD_PC refers to a got entry. You might want to rename this function.
https://reviews.llvm.org/D31792
>
> > // If the relocation points to something in the file, we can
> process it.
> > - bool Constant =
> > + bool IsConstant =
> > isStaticLinkTimeConstant<ELFT>(Expr, Type, Body, Sec,
> Rel.r_offset);
>
> In the future please do renames like these in their own patch, as it
> make each patch easier to read.
>
I'll keep that in mind.
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170406/3555cdf7/attachment.html>
More information about the llvm-commits
mailing list