[PATCH] D43920: Set DF_TEXTREL instead of creating an empty DT_TEXTREL.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 1 14:21:42 PST 2018


On Thu, Mar 1, 2018 at 2:19 PM Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> Sigh, looks like DF_TEXTREL is also missing from musl.
>
> Maybe we could decide on DF_TEXTREL or DT_TEXTREL based on
> -enable-new-dtags?
>

Hmm, I could, but I'd just do both then for the sake of simplicity.


> Cheers,
> Rafael
>
> Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:
>
> > ruiu added a comment.
> >
> > GNU linkers seem to emit both since perhaps almost 20 years ago. I'm
> actually fine with emitting both, but since we've had such an extremely
> long transition period, I'd like to emit just DF_TEXTREL if all OSes are OK
> with that.
> >
> >
> > https://reviews.llvm.org/D43920
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180301/9285eed5/attachment.html>


More information about the llvm-commits mailing list