This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [AArch64] Don't generate GOT entry for large model TLS LE relocation
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: Jiong Wang <jiong dot wang at foss dot arm dot com>
- Cc: "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Wed, 23 Dec 2015 18:10:38 +0000
- Subject: Re: [AArch64] Don't generate GOT entry for large model TLS LE relocation
- Authentication-results: sourceware.org; auth=none
- References: <567AD20C dot 4040007 at foss dot arm dot com>
On 23 December 2015 at 16:55, Jiong Wang <jiong.wang@foss.arm.com> wrote:
> This patch revert the following commit
> commit b7a944fea3a0194d81f6de4d958f3a1d2c6ad03a
> Author: Renlin Li <renlin.li@arm.com>
> Date: Fri Oct 2 17:02:53 2015 +0100
I have not looked at this in detail, I'm out of the office at the
moment, but I did notice that this is more than a revert!
Please split this into two patches.
1) The revert request.
2) The new functionality you want to add.
I think the revert is safe because it is a relaxation, leaving the
original sequence in place should be fine, also it only affects large
model and the default model is small. I think the patch being reverted
here may have had test cases submitted later as a separate patch?
The revert part of this is IMO going to be suitable for 2.26, however
I don;t think we should be taking the new functionality part on 2.26,
at least not until it has baked on the trunk for a while.
Cheers
/Marcus