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: breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"

From: Jeff Law <law at redhat dot com>
To: Steven Bosscher <stevenb dot gcc at gmail dot com>, Segher Boessenkool <segher at kernel dot crashing dot org>
Cc: Hans-Peter Nilsson <hp at bitrange dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 09 Apr 2015 08:15:44 -0600
Subject: Re: breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"
Authentication-results: sourceware.org; auth=none
References: <04f427aaf952283357bdc5cdc0bdabee27832d20 dot 1428528396 dot git dot segher at kernel dot crashing dot org> <alpine dot BSF dot 2 dot 02 dot 1504082222420 dot 69949 at arjuna dot pair dot com> <20150409124117 dot GA24625 at gate dot crashing dot org> <CABu31nN9yjnZBk4bEcjCx=7U8Y5hqdCybmiCuyv_a7pb9aD52A at mail dot gmail dot com>

On 04/09/2015 07:21 AM, Steven Bosscher wrote:

On Thu, Apr 9, 2015 at 2:41 PM, Segher Boessenkool wrote:

It would be nice if there would be some cc0 target in the compile farm,
since cc0 isn't going away any time soon :-(


In this case it would be enough to replace the "#ifndef/#ifdef
HAVE_cc0" code with "if (HAVE_cc0)".

That's the simplest way to avoid compile breakage. Likewise for so
many other #ifdef code (HAVE_conditional_move, HAVE_trap, etc.).

Perhaps something to work on in the next stage1...

Most definitely a direction I want to see us moving. The glibc projectrecently went through the pain of the transition, but I believe it'll beworth it long term for them and even more so for us (we have a lot moreconditionally compiled code than glibc).


jeff

References:
- [PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)
  - From: Segher Boessenkool
- breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"
  - From: Hans-Peter Nilsson
- Re: breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"
  - From: Segher Boessenkool
- Re: breakage with "[PATCH] combine: Disregard clobbers in another test for two SETs (PR65693)"
  - From: Steven Bosscher

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