From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32359 invoked by alias); 23 Aug 2011 15:46:47 -0000 Received: (qmail 32351 invoked by uid 22791); 23 Aug 2011 15:46:47 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 23 Aug 2011 15:46:33 +0000 Received: (qmail 7141 invoked from network); 23 Aug 2011 15:46:32 -0000 Received: from unknown (HELO ?84.152.222.249?) (bernds@127.0.0.2) by mail.codesourcery.com with ESMTPA; 23 Aug 2011 15:46:32 -0000 Message-ID: <4E53CAFE.1050403@codesourcery.com> Date: Tue, 23 Aug 2011 16:16:00 -0000 From: Bernd Schmidt User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.18) Gecko/20110801 Lightning/1.0b3pre Thunderbird/3.1.11 MIME-Version: 1.0 To: Jakub Jelinek CC: Eric Botcazou , Richard Sandiford , Richard Henderson , GCC Patches 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> In-Reply-To: <20110823154224.GJ2687@tyan-ft48-01.lab.bos.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-08/txt/msg01911.txt.bz2 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