This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 1/2] Remove the IPA-SRA call to build_ref_for_offset
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Martin Jambor <mjambor at suse dot cz>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Guenther <rguenther at suse dot de>
- Date: Mon, 6 Dec 2010 10:56:14 -0800
- Subject: Re: [PATCH 1/2] Remove the IPA-SRA call to build_ref_for_offset
- References: <20100908164332.622577593@virgil.suse.cz> <20100908164349.353922665@virgil.suse.cz> <AANLkTinej4nLRRfrM8=xKiv31FGaUQf5ynG+MFJyLCrC@mail.gmail.com>
On Fri, Sep 10, 2010 at 5:21 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Wed, Sep 8, 2010 at 9:43 AM, Martin Jambor <mjambor@suse.cz> wrote:
>> Hi,
>>
>> this is the same patch I sent here in August:
>> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00609.html
>>
>> It reimplements parameter alterations in callers of a function being
>> modified by IPA-SRA so that it always produces MEM_REF relying on
>> force_gimple_operand_gsi to break up refs if necessary (e.g. if we
>> have a mem_ref of an addr_expr of an array_ref with variable index of
>> another mem_ref etc...).
>>
>> I have bootstrapped and tested this patch separately on x86_64-linux.
>> OK for trunk?
>>
>> Thanks,
>>
>> Martin
>>
>>
>> 2010-08-08 ?Martin Jambor ?<mjambor@suse.cz>
>>
>> ? ? ? ?PR tree-optimization/44972
>> ? ? ? ?* ipa-prop.c (ipa_modify_call_arguments): Build MEM_REF instead of
>> ? ? ? ?calling build_ref_for_offset.
>>
>> ? ? ? ?* testsuite/g++.dg/torture/pr34850.C: Remove expected warning.
>>
>
> This caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45644
>
This also caused:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46823
--
H.J.