This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Fix ICE on unaligned record field
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Tue, 3 Mar 2015 10:34:08 -0800
- Subject: Re: [patch] Fix ICE on unaligned record field
- Authentication-results: sourceware.org; auth=none
- References: <3887233 dot jQSJCmu5YU at polaris> <CAFiYyc1S==cr8Aq_YxCmFfcCDE21eH8AnO4n1Ri8F-K7_y8_yg at mail dot gmail dot com> <20141203140207 dot GL8214 at virgil dot suse> <14069807 dot eYpOW1jBVs at polaris> <20150225154303 dot GE20953 at virgil dot suse>
On Wed, Feb 25, 2015 at 7:43 AM, Martin Jambor <mjambor@suse.cz> wrote:
> Hi Eric and Richard,
>
> On Tue, Jan 06, 2015 at 06:07:12PM +0100, Eric Botcazou wrote:
>> Martin,
>>
>> > I suppose that could be done by something like the following, which I
>> > have tested only very mildly so far, in particular I have not double
>> > checked that get_inner_reference is cfun-agnostic.
>>
>> The patch introduces no regressions on x86-64/Linux and makes the testcase
>> (gnat.dg/specs/pack12.ads attached to the first message) pass.
>>
>> Do you plan to install it (along with the testcase)?
>>
>
> for various reasons I was not able to do it earlier, but today I have
> re-bootstrapped the following (the only change is the added testcase)
> on x86_64-linux and it passes OK. Should I commit it to trunk then?
>
> Thanks,
>
> Martin
>
>
> 2015-02-25 Martin Jambor <mjambor@suse.cz>
> Eric Botcazou <ebotcazou@adacore.com>
>
> gcc/
> * tree-sra.c (ipa_sra_check_caller_data): New type.
> (has_caller_p): Removed.
> (ipa_sra_check_caller): New function.
> (ipa_sra_preliminary_function_checks): Use it.
>
This caused:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65305
--
H.J.