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: [PATCH] For FFS/CLZ/CTZ/CLRSB/POPCOUNT/PARITY/BSWAP require operand mode equal to operation mode (or VOIDmode) (PR middle-end/50161)

From: Bernd Schmidt <bernds at codesourcery dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Eric Botcazou <ebotcazou at adacore dot com>, Richard Sandiford <richard dot sandiford at linaro dot org>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Tue, 23 Aug 2011 17:45:02 +0200
Subject: Re: [PATCH] For FFS/CLZ/CTZ/CLRSB/POPCOUNT/PARITY/BSWAP require operand mode equal to operation mode (or VOIDmode) (PR middle-end/50161)
References: <4DF9FA9A.8040505@codesourcery.com> <4DFA2E85.2030601@redhat.com> <4DFFA1AE.7070405@codesourcery.com> <20110823090549.GC2687@tyan-ft48-01.lab.bos.redhat.com> <4E53744B.6040502@codesourcery.com> <20110823095243.GD2687@tyan-ft48-01.lab.bos.redhat.com> <4E5379A6.1020905@codesourcery.com> <20110823120605.GE2687@tyan-ft48-01.lab.bos.redhat.com> <20110823154224.GJ2687@tyan-ft48-01.lab.bos.redhat.com>

On 08/23/11 17:42, Jakub Jelinek wrote:
> On Tue, Aug 23, 2011 at 02:06:05PM +0200, Jakub Jelinek wrote:
>> We can remove that restriction again once CONST_INTs are no longer VOIDmode.
>>
>> Here is an untested patch, will bootstrap/regtest it now on x86_64-linux
>> and i686-linux, on c6x it should make no difference IMHO (looked like a typo
>> in the expander which wasn't used anyway), can somebody test it on AVR and
>> BFIN?  My grepping through *.md didn't find any other places where the
>> operand wouldn't have the same mode as operation.
> 
> Now successfully bootstrapped/regtested on x86_64-linux and i686-linux
> and tested on AVR by Georg-Johann, ok for trunk?

Ok. Will fix up bfin/c6x if necessary.


Bernd

References:
- Re: Add __builtin_clrsb, similar to clz/ctz
  - From: Jakub Jelinek
- Re: Add __builtin_clrsb, similar to clz/ctz
  - From: Bernd Schmidt
- Re: Add __builtin_clrsb, similar to clz/ctz
  - From: Jakub Jelinek
- Re: Add __builtin_clrsb, similar to clz/ctz
  - From: Bernd Schmidt
- [PATCH] For FFS/CLZ/CTZ/CLRSB/POPCOUNT/PARITY/BSWAP require operand mode equal to operation mode (or VOIDmode) (PR middle-end/50161)
  - From: Jakub Jelinek
- Re: [PATCH] For FFS/CLZ/CTZ/CLRSB/POPCOUNT/PARITY/BSWAP require operand mode equal to operation mode (or VOIDmode) (PR middle-end/50161)
  - From: Jakub Jelinek

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