From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6422 invoked by alias); 18 Aug 2007 07:54:17 -0000 Received: (qmail 6327 invoked by uid 22791); 18 Aug 2007 07:54:17 -0000 X-Spam-Check-By: sourceware.org Received: from fencepost.gnu.org (HELO fencepost.gnu.org) (140.186.70.10) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 18 Aug 2007 07:54:15 +0000 Received: from monty-python.gnu.org ([199.232.76.173]) by fencepost.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1IMJC7-00045t-Ho for gcc-patches@gnu.org; Sat, 18 Aug 2007 03:58:03 -0400 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1IMJ8N-0005Dh-Vx for gcc-patches@gnu.org; Sat, 18 Aug 2007 03:54:12 -0400 Received: from mtagate1.de.ibm.com ([195.212.29.150]) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1IMJ8N-0005D4-It for gcc-patches@gnu.org; Sat, 18 Aug 2007 03:54:11 -0400 Received: from d12nrmr1607.megacenter.de.ibm.com (d12nrmr1607.megacenter.de.ibm.com [9.149.167.49]) by mtagate1.de.ibm.com (8.13.8/8.13.8) with ESMTP id l7I7s6S8108056 for ; Sat, 18 Aug 2007 07:54:06 GMT Received: from d12av04.megacenter.de.ibm.com (d12av04.megacenter.de.ibm.com [9.149.165.229]) by d12nrmr1607.megacenter.de.ibm.com (8.13.8/8.13.8/NCO v8.5) with ESMTP id l7I7s6XG1998930 for ; Sat, 18 Aug 2007 09:54:06 +0200 Received: from d12av04.megacenter.de.ibm.com (loopback [127.0.0.1]) by d12av04.megacenter.de.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id l7I7s656029309 for ; Sat, 18 Aug 2007 09:54:06 +0200 Received: from d12mc102.megacenter.de.ibm.com (d12mc102.megacenter.de.ibm.com [9.149.167.114]) by d12av04.megacenter.de.ibm.com (8.12.11.20060308/8.12.11) with ESMTP id l7I7s6ZI029306; Sat, 18 Aug 2007 09:54:06 +0200 In-Reply-To: Subject: Re: [PATCH] [4.3 projects] Verctorizer - versioning for alias To: Victor Kaplansky Cc: gcc-patches@gnu.org, rakdver@kam.mff.cuni.cz X-Mailer: Lotus Notes Release 7.0 HF277 June 21, 2006 Message-ID: From: Dorit Nuzman Date: Sat, 18 Aug 2007 07:54:00 -0000 MIME-Version: 1.0 Content-type: text/plain; charset=US-ASCII X-Detected-Kernel: Linux 2.0.3x (2) X-IsSubscribed: yes 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: 2007-08/txt/msg01175.txt.bz2 Victor Kaplansky/Haifa/IBM wrote on 16/08/2007 16:48:11: > Dorit Nuzman/Haifa/IBM wrote on 16/08/2007 04:11:46 PM: > > ... > > I think you can use TYPE_VECTOR_SUBPARTS() instead of > > GET_MODE_SIZE (TYPE_MODE ()) (just a tiny bit more compact) > > I think that it slightly different. For example TYPE_VECTOR_SUBPARTS() > returns 4 for a vector of integers on ppc, while > GET_MODE_SIZE (TYPE_MODE ()) returns 16 - size of vector in bytes. > in that case you can use TYPE_SIZE_UNIT () dorit > > > > Also, for more compact code, you could compute the vf*step once and > > reuse it for both then and else. For your consideration. > > > > + (vinfo_for_stmt (DR_STMT (dr)))))); > > + > > + segment_length = > > + fold_convert (sizetype, > > + fold_build2 (PLUS_EXPR, integer_type_node, > > + fold_build2 (MULT_EXPR, integer_type_node, DR_STEP (dr), > > + vect_factor), > > + vector_size)); > > + > > + > > + } > > + else > > + { > > + segment_length = > > + fold_convert (sizetype, > > + fold_build2 (MULT_EXPR, integer_type_node, DR_STEP (dr), > > + vect_factor)); > > + } > > + > > + return segment_length; > > +} > > OK, I'll address this in next patch.