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 [3/4]

From: Bernd Schmidt <bernds at codesourcery dot com>
To: Steven Bosscher <stevenb dot gcc at gmail dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 11 Sep 2014 18:06:28 +0200
Subject: Re: ptx preliminary rtl patches [3/4]
Authentication-results: sourceware.org; auth=none
References: <5411A31E dot 3030207 at codesourcery dot com> <CABu31nMfsZCSOJyRJAjT6+dZ28rdd=5LGaOZRF=qRFVWOvWvmw at mail dot gmail dot com>

On 09/11/2014 05:55 PM, Steven Bosscher wrote:

On Thu, Sep 11, 2014 at 3:26 PM, Bernd Schmidt wrote:

nvptx will be the first port to use BImode and have STORE_FLAG_VALUE==-1.
That has exposed a bug in combine where we can end up calling
num_sign_bit_copies for a BImode value. However, the return value is always
1 in that case, so it doesn't tell us anything and is going to be
misinterpreted by the caller.

Bootstrapped and tested on x86_64-linux, together with the other patches.
Ok?


This should be handled in num_sign_bit_copies itself, i.e. handle BImode there.

What do you expect that function to do different? It returns the correctvalue.



Bernd

Follow-Ups:
- Re: ptx preliminary rtl patches [3/4]
  - From: Steven Bosscher

References:
- ptx preliminary rtl patches [3/4]
  - From: Bernd Schmidt
- Re: ptx preliminary rtl patches [3/4]
  - From: Steven Bosscher

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