This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH v2, rs6000] Fix PR72827 (ada bootstrap failure)
- From: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, ebotcazou at adacore dot com
- Date: Thu, 1 Sep 2016 09:44:35 -0500
- Subject: Re: [PATCH v2, rs6000] Fix PR72827 (ada bootstrap failure)
- Authentication-results: sourceware.org; auth=none
- References: <83ced7a8-d832-87d8-aaf6-03a55300af62@linux.vnet.ibm.com> <20160831061954.GB21601@gate.crashing.org> <04b57aff-e788-012f-cc65-2f8356ad7cc7@linux.vnet.ibm.com> <20160901143231.GB9816@gate.crashing.org>
On Sep 1, 2016, at 9:32 AM, Segher Boessenkool <segher@kernel.crashing.org> wrote:
>
> On Wed, Aug 31, 2016 at 09:15:32PM -0500, Bill Schmidt wrote:
>> This patch (suggested by Michael Meissner) instead prevents the problem
>> by disallowing reg+reg addressing for TImode, allowing D-form addressing
>> to be used for the separate stores of the GPRs. This is not an ideal
>> permanent solution, because it disallows reg+reg addressing not only for
>> TImode in GPRs, but also for TImode in VSRs. Our intent is to work on a
>> solution to provide a scratch register via secondary reload, but this
>> may take some time. Thus I'd like to submit this patch as a short-term
>> solution for the bootstrap problem until then.
>>
>> I plan to do a performance evaluation of this patch for informational
>> purposes, but I don't expect much change, and the results won't affect
>> our choice to handle this properly in secondary reload.
>>
>> Bootstrapped and tested on powerpc64le-unknown-linux-gnu with no
>> regressions, and the gnattools now build properly so that the ada
>> bootstrap succeeds. Is this ok for trunk?
>
> Yes please. Thanks,
r239938, thanks!
Bill
>
>
> Segher
>