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: ptx preliminary rtl patches [1/4]

From: Bernd Schmidt <bernds at codesourcery dot com>
To: Segher Boessenkool <segher at kernel dot crashing dot org>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 31 Oct 2014 18:22:43 +0100
Subject: Re: ptx preliminary rtl patches [1/4]
Authentication-results: sourceware.org; auth=none
References: <5411A2A6 dot 1030303 at codesourcery dot com> <20141030173615 dot GA22655 at gate dot crashing dot org>

On 10/30/2014 06:36 PM, Segher Boessenkool wrote:

On Thu, Sep 11, 2014 at 03:24:54PM +0200, Bernd Schmidt wrote:

The nvptx backend is somewhat unusual in that call insns set a pseudo.
The combiner is surprised by this and allows combining them into other
insns, which remain as INSN rather than CALL_INSN. Aborts ensue.


distribute_notes has code for when I2 is a CALL.  This suggests that
on some other targets there can be useful combinations made, so it
isn't in that case terribly nice to disable all combinations with CALLs
as I0 or I1 or I2.

Or it's dead code :-)

I suspect the latter. As far as I can tell the patch does not changecode generation on x86_64 at least (on a large set of input filesincluding gcc, spec2k and linux-kernel).



Bernd

References:
- Re: ptx preliminary rtl patches [1/4]
  - From: Segher Boessenkool

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