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: [0/3] Fix PR78120, in ifcvt/rtlanal/i386.

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Segher Boessenkool <segher at kernel dot crashing dot org>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 24 Nov 2016 15:26:45 +0100
Subject: Re: [0/3] Fix PR78120, in ifcvt/rtlanal/i386.
Authentication-results: sourceware.org; auth=none
References: <e6d06b7a-831b-299a-5c5a-507e267238a4@redhat.com> <2bde94e0-b470-8aad-6a9c-91e45dc8b687@redhat.com> <20161124142119.GD14394@gate.crashing.org>

On 11/24/2016 03:21 PM, Segher Boessenkool wrote:

Combine uses insn_rtx_cost extensively.  I have tried to change it to use
the full rtx cost, not just the source cost, a few times before, and it
always only regressed.  Part of it is that most ports' cost calculations
are, erm, not so great -- every target we care about needs fixes.

Let's please not try this in stage 3.

It got approved and committed. Do you want me to revert it now or waitfor obvious signs of fallout?



Bernd

Follow-Ups:
- Re: [0/3] Fix PR78120, in ifcvt/rtlanal/i386.
  - From: Segher Boessenkool

References:
- [0/3] Fix PR71280, in ifcvt/rtlanal/i386.
  - From: Bernd Schmidt
- Re: [0/3] Fix PR78120, in ifcvt/rtlanal/i386.
  - From: Bernd Schmidt
- Re: [0/3] Fix PR78120, in ifcvt/rtlanal/i386.
  - From: Segher Boessenkool

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