This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix gimple-fold
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Martin LiÅka <mliska at suse dot cz>
- Cc: Jakub Jelinek <jakub at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 18 Mar 2014 16:01:57 +0100
- Subject: Re: [PATCH] Fix gimple-fold
- Authentication-results: sourceware.org; auth=none
- References: <53283E3B dot 4050700 at suse dot cz> <CAFiYyc3TBe-zHcOaMooZQNxN6j9L+pDuxHq_=KpyAoUnYG1THg at mail dot gmail dot com> <20140318131316 dot GU22862 at tucnak dot redhat dot com> <53284A3A dot 9020105 at suse dot cz> <CAFiYyc0qK7HwJJ6AOYZS3EvMVHk-Jh2Gs_Y6B7=RoF7DrDujSQ at mail dot gmail dot com> <53285F38 dot 7030302 at suse dot cz>
On Tue, Mar 18, 2014 at 3:59 PM, Martin LiÅka <mliska@suse.cz> wrote:
> Patch passes bootstrap and regtest.
>
> I fixed indentation according to discussion with Jakub.
>
> OK for trunk?
Ok.
Thanks,
Richard.
> Thanks,
> Martin
>
>
> On 03/18/2014 02:55 PM, Richard Biener wrote:
>>
>> On Tue, Mar 18, 2014 at 2:29 PM, Martin LiÅka <mliska@suse.cz> wrote:
>>>
>>> Thank you for feedback,
>>
>> Ok if it passes bootstrap / regtest.
>>
>> Thanks,
>> Richard.
>>
>>> new changelog:
>>>
>>> 2014-03-18 Martin Liska <mliska@suse.cz>
>>>
>>> * cgraph.c (cgraph_update_edges_for_call_stmt_node): Added case
>>> when
>>> gimple call statement is update.
>>>
>>> * gimple-fold.c (gimple_fold_call): Changed order for
>>> GIMPLE_ASSIGN
>>> and
>>> GIMPLE_CALL, where gsi iterator still points to GIMPLE CALL.
>>>
>>> OK for trunk?
>>>
>>> Martin
>>>
>>>
>>>
>>> On 03/18/2014 02:13 PM, Jakub Jelinek wrote:
>>>>
>>>> Hi!
>>>>
>>>>>> 2014-03-18 Martin Liska <mliska@suse.cz>
>>>>>>
>>>>>> * cgraph.c (cgraph_update_edges_for_call_stmt_node): added
>>>>>> case
>>>>>> when
>>>>>> gimple call statement is updated.
>>>>
>>>> Capital letter after :
>>>>
>>>>>> * gimple-fold.c (gimple_fold_call): changed order for
>>>>>> GIMPLE_ASSIGN and
>>>>
>>>> Likewise here.
>>>>
>>>> Jakub
>>>
>>>
>