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: IRA costs tweaks, PR 56069

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Vladimir Makarov <vmakarov at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Jakub Jelinek <jakub at redhat dot com>
Date: Mon, 25 Apr 2016 11:11:28 +0200
Subject: Re: IRA costs tweaks, PR 56069
Authentication-results: sourceware.org; auth=none
References: <56D71977 dot 5040204 at t-online dot de> <56D72316 dot 2060207 at redhat dot com> <56D760CF dot 4030308 at redhat dot com>

On 03/02/2016 10:53 PM, Vladimir Makarov wrote:

The patch is ok for me.  But I'd wait for GCC7.  People are sensitive to
their code performance degradation.  Even in most cases the patch
improves performance, in some case it can worsen code and people might
fill new PRs after such change.

I've retested now, and it turns out that it turns two guality/sra-1.ctests into UNSUPPORTED. Apparently a variable is now destructivelyoverwritten. Is that something we worry about (I'm inclined to think no)?



Bernd

Follow-Ups:
- Re: IRA costs tweaks, PR 56069
  - From: Jeff Law

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