From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15526 invoked by alias); 28 Sep 2004 18:54:21 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 15519 invoked from network); 28 Sep 2004 18:54:19 -0000 Received: from unknown (HELO dns0.mips.com) (209.232.97.206) by sourceware.org with SMTP; 28 Sep 2004 18:54:19 -0000 Received: from mercury.mips.com (sbcns-dmz [209.232.97.193]) by dns0.mips.com (8.12.11/8.12.11) with ESMTP id i8SIs7h0017791; Tue, 28 Sep 2004 11:54:07 -0700 (PDT) Received: from exchange.MIPS.COM (exchange [192.168.20.29]) by mercury.mips.com (8.12.11/8.12.11) with ESMTP id i8SIs74R009768; Tue, 28 Sep 2004 11:54:07 -0700 (PDT) Received: from pcfu ([192.168.20.169]) by exchange.MIPS.COM with Microsoft SMTPSVC(6.0.3790.0); Tue, 28 Sep 2004 11:54:06 -0700 Message-ID: <002001c4a58c$878ff6e0$a914a8c0@MIPS.COM> Reply-To: "Chao-ying Fu" From: "Chao-ying Fu" To: "Dorit Naishlos" Cc: "Richard Henderson" , "Jim Wilson" , , "Stephens, Nigel" , "Thekkath, Radhika" , "Richard Sandiford" , "Uhler, Mike" References: Subject: Re: [patch] extend.texi MIPS PS/3D Support Date: Tue, 28 Sep 2004 19:35:00 -0000 Organization: MIPS Technologies, Inc. MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 28 Sep 2004 18:54:06.0115 (UTC) FILETIME=[87896730:01C4A58C] X-Scanned-By: MIMEDefang 2.39 X-SW-Source: 2004-09/txt/msg02891.txt.bz2 Hello, build_vector_type() -> make_vecotr_type() -> layout_type() returns DImode for 2 units of SImode (innermode). Then, vect_analyze_operatoions()->vectorizable_operation() checks if optab->handlers[DImode].insn_code is available. But, we should check optab->handlers[V2SImode].insn_code. Regards, Chao-ying ----- Original Message ----- From: "Dorit Naishlos" To: "Richard Henderson" Cc: "Chao-ying Fu" ; ; "Stephens, Nigel" ; "Thekkath, Radhika" ; "Richard Sandiford" ; "Uhler, Mike" ; "Jim Wilson" Sent: Tuesday, September 28, 2004 6:25 AM Subject: Re: [patch] extend.texi MIPS PS/3D Support > > > The real problem is that you're pulling out SImode and deciding > > that add_optab[SImode] is the vector addition operation. > > but how come we get SImode here? we call build_vector_type with > innertype = SI, and > nunits = UNITS_PER_SIMD_WORD / nbytes = 8 / 4 = 2 > so it should return a V2SI vectype, which is not supported by the target > and therefore should have a BLKmode. no? > > dorit > > > > > Richard Henderson > To: Dorit Naishlos/Haifa/IBM@IBMIL > cc: Chao-ying Fu , gcc-patches@gcc.gnu.org, "Stephens, Nigel" > 28/09/2004 01:43 , "Thekkath, Radhika" , Richard > Sandiford , "Uhler, Mike" , Jim > Wilson > Subject: Re: [patch] extend.texi MIPS PS/3D Support > > > > > > On Tue, Sep 28, 2004 at 01:30:00AM +0200, Dorit Naishlos wrote: > > Maybe the problem is here (in get_vectype_for_scalar_type): > > > > vectype = build_vector_type (scalar_type, nunits); > > if (TYPE_MODE (vectype) == BLKmode) > > return NULL_TREE; > > > > Maybe the vectype that is built has a mode other than BLKmode, although > > it's not supported by the target? > > Yes, e.g. V4QI may be represented with SImode. But that's fine, > since the vectorizer may be able to optimize data movement loops > with no other vector support in the target. (E.g. if target and > destination are alignable.) > > The real problem is that you're pulling out SImode and deciding > that add_optab[SImode] is the vector addition operation. > > > r~ > > > >