From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19775 invoked by alias); 29 Sep 2011 05:39:56 -0000 Received: (qmail 19284 invoked by uid 22791); 29 Sep 2011 05:39:54 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from mail-iy0-f175.google.com (HELO mail-iy0-f175.google.com) (209.85.210.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 29 Sep 2011 05:39:38 +0000 Received: by iadx2 with SMTP id x2so366197iad.20 for ; Wed, 28 Sep 2011 22:39:37 -0700 (PDT) MIME-Version: 1.0 Received: by 10.231.82.131 with SMTP id b3mr14228687ibl.74.1317274777550; Wed, 28 Sep 2011 22:39:37 -0700 (PDT) Received: by 10.231.205.207 with HTTP; Wed, 28 Sep 2011 22:39:37 -0700 (PDT) In-Reply-To: <20110928215104.GA29339@bromo.med.uc.edu> References: <20110715232425.GA24793@atrey.karlin.mff.cuni.cz> <20110928133105.GA26045@bromo.med.uc.edu> <20110928215104.GA29339@bromo.med.uc.edu> Date: Thu, 29 Sep 2011 08:13:00 -0000 Message-ID: Subject: Re: Use of vector instructions in memmov/memset expanding From: Michael Zolotukhin To: Jack Howarth Cc: gcc-patches@gcc.gnu.org, Jan Hubicka , Richard Guenther , "H.J. Lu" , izamyatin@gmail.com, areg.melikadamyan@gmail.com Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable 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-09/txt/msg01908.txt.bz2 > Michael, > Did you bootstrap with --enable-checking=3Dyes? I am seeing the bootst= rap > failure... I checked bootstrap, specs and 'make check' with the complete patch. Separate patches for ME and BE were only tested for build (no bootstrap) and 'make check'. I think it's better to apply the complete patch, but review the separate patches (to make it easier). > ps There also seems to be common sections in the memfunc-mid.patch and me= mfunc-be.patch patches. That's true, some new routines from middle-end are used in back-end changes - I couldn't separate the patches in other way without significant changes in them. On 29 September 2011 01:51, Jack Howarth wrote: > On Wed, Sep 28, 2011 at 05:33:23PM +0400, Michael Zolotukhin wrote: >> > =A0 It appears that part 1 of the patch wasn't really attached. >> Thanks, resending. > > Michael, > =A0 =A0Did you bootstrap with --enable-checking=3Dyes? I am seeing the bo= otstrap > failure... > > /sw/src/fink.build/gcc47-4.7.0-1/darwin_objdir/./prev-gcc/g++ -B/sw/src/f= ink.build/gcc47-4.7.0-1/darwin_objdir/./prev-gcc/ -B/sw/lib/gcc4.7/x86_64-a= pple-darwin11.2.0/bin/ -nostdinc++ -B/sw/src/fink.build/gcc47-4.7.0-1/darwi= n_objdir/prev-x86_64-apple-darwin11.2.0/libstdc++-v3/src/.libs -B/sw/src/fi= nk.build/gcc47-4.7.0-1/darwin_objdir/prev-x86_64-apple-darwin11.2.0/libstdc= ++-v3/libsupc++/.libs -I/sw/src/fink.build/gcc47-4.7.0-1/darwin_objdir/prev= -x86_64-apple-darwin11.2.0/libstdc++-v3/include/x86_64-apple-darwin11.2.0 -= I/sw/src/fink.build/gcc47-4.7.0-1/darwin_objdir/prev-x86_64-apple-darwin11.= 2.0/libstdc++-v3/include -I/sw/src/fink.build/gcc47-4.7.0-1/gcc-4.7-2011092= 7/libstdc++-v3/libsupc++ -L/sw/src/fink.build/gcc47-4.7.0-1/darwin_objdir/p= rev-x86_64-apple-darwin11.2.0/libstdc++-v3/src/.libs -L/sw/src/fink.build/g= cc47-4.7.0-1/darwin_objdir/prev-x86_64-apple-darwin11.2.0/libstdc++-v3/libs= upc++/.libs -c =A0 -g -O2 -mdynamic-no-pic -gtoggle -DIN_GCC =A0 -W -Wall -= Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-l= ong -Wno-variadic-macros -Wno-overlength-strings -Werror -fno-common =A0-DH= AVE_CONFIG_H -I. -I. -I../../gcc-4.7-20110927/gcc -I../../gcc-4.7-20110927/= gcc/. -I../../gcc-4.7-20110927/gcc/../include -I../../gcc-4.7-20110927/gcc/= ../libcpp/include -I/sw/include -I/sw/include =A0-I../../gcc-4.7-20110927/g= cc/../libdecnumber -I../../gcc-4.7-20110927/gcc/../libdecnumber/dpd -I../li= bdecnumber -I/sw/include =A0-I/sw/include -DCLOOG_INT_GMP -DCLOOG_ORG -I/sw= /include ../../gcc-4.7-20110927/gcc/emit-rtl.c -o emit-rtl.o > ../../gcc-4.7-20110927/gcc/emit-rtl.c: In function =91rtx_def* adjust_add= ress_1(rtx, machine_mode, long int, int, int)=92: > ../../gcc-4.7-20110927/gcc/emit-rtl.c:2060:26: error: unused variable =91= max_align=92 [-Werror=3Dunused-variable] > cc1plus: all warnings being treated as errors > > on x86_64-apple-darwin11 with your patches. > =A0 =A0 =A0 =A0 =A0Jack > ps There also seems to be common sections in the memfunc-mid.patch and me= mfunc-be.patch patches. > --=20 --- Best regards, Michael V. Zolotukhin, Software Engineer Intel Corporation.