This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH PR64434]

From: Yuri Rumyantsev <ysrumyan at gmail dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>, Igor Zamyatin <izamyatin at gmail dot com>
Date: Wed, 14 Jan 2015 13:32:13 +0300
Subject: Re: [PATCH PR64434]
Authentication-results: sourceware.org; auth=none
References: <CAEoMCqR7iJ3kt+G8r95B44EtQG1VDe2a2+tg2ejTv7jqNeo5Fw at mail dot gmail dot com> <20150114102338 dot GL1405 at tucnak dot redhat dot com>

Sorry, I resend correct patch.

Yuri.

2015-01-14 13:23 GMT+03:00 Jakub Jelinek <jakub@redhat.com>:
> On Wed, Jan 14, 2015 at 01:12:20PM +0300, Yuri Rumyantsev wrote:
>> Hi All,
>>
>> Here is updated patch which was redesigned accordingly to Richard review.
>> It performs swapping operands of commutative operations to expand the
>> expensive one first.
>>
>> Bootstrap and regression testing did not show any new failures.
>>
>> Is it OK for trunk?
>
> Haven't you just reposted the patch from December?  I don't see any changes
> from then...
>
>> gcc/ChangeLog
>> 2015-01-14  Yuri Rumyantsev  <ysrumyan@gmail.com>
>>
>> PR tree-optimization/64434
>> * cfgexpand.c (reorder_operands): New function.
>> (expand_gimple_basic_block): Insert call of reorder_operands.
>>
>> gcc/testsuite/ChangeLog
>> * gcc.dg/torture/pr64434.c: New test.
>
>         Jakub

Attachment: patch
Description: Binary data

Follow-Ups:
- Re: [PATCH PR64434]
  - From: Jakub Jelinek

References:
- [PATCH PR64434]
  - From: Yuri Rumyantsev
- Re: [PATCH PR64434]
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]