From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15556 invoked by alias); 13 Nov 2008 21:20:39 -0000 Received: (qmail 15546 invoked by uid 22791); 13 Nov 2008 21:20:38 -0000 X-Spam-Check-By: sourceware.org Received: from caip.rutgers.edu (HELO caip.rutgers.edu) (128.6.236.16) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 13 Nov 2008 21:19:35 +0000 Received: from caipclassic.rutgers.edu (caipclassic.rutgers.edu [128.6.237.54]) by caip.rutgers.edu (8.13.8/8.13.8) with ESMTP id mADLJS9s016873 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 13 Nov 2008 16:19:28 -0500 Date: Thu, 13 Nov 2008 21:52:00 -0000 From: "Kaveh R. GHAZI" To: Richard Guenther cc: Ralph Loader , gcc-patches@gcc.gnu.org, ubizjak@gmail.com Subject: Re: [PATCH] Fix for PR 37809 and 37807 In-Reply-To: <84fc9c000811050120u27692d32ra32c27d1eefc2cdf@mail.gmail.com> Message-ID: References: <20081022181234.2acf7548@i.geek.nz> <84fc9c000810220316t20257102t9ce3b53e95d3400b@mail.gmail.com> <20081105205605.20d170c4@i.geek.nz> <84fc9c000811050120u27692d32ra32c27d1eefc2cdf@mail.gmail.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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: 2008-11/txt/msg00603.txt.bz2 On Wed, 5 Nov 2008, Richard Guenther wrote: > > I checked this and gen_lowpart instead of gen_lowpart_or_truncate > > appears fine. > > > > Attached is patch modified to use gen_lowpart, and with the test case > > cleaned up & changeloged. > > The patch is ok this way. > > Thanks, > Richard. > > >> > 2008-10-19 Ralph Loader > >> > > >> > PR middle-end/37807, middle-end/37809 > >> > * combine.c (force_to_mode): Do not process vector types. > >> > > >> > * rtlanal.c (nonzero_bits1): Do not process vector types. > >> > (num_sign_bit_copies1): Likewise. The testcase mmx-8.c crashes on the 4.3/4.2 branches, e.g. http://gcc.gnu.org/ml/gcc-testresults/2008-11/msg01056.html Was the patch tested on the branches prior to installation? --Kaveh -- Kaveh R. Ghazi ghazi@caip.rutgers.edu