From: Jack Howarth <howarth@bromo.med.uc.edu>
To: Sebastian Pop <sebpop@gmail.com>
Cc: gcc-patches@gcc.gnu.org, rguenther@suse.de,
gcc-graphite@googlegroups.com
Subject: Re: [PATCH 0/5] Remove the LAMBDA framework
Date: Tue, 18 Jan 2011 19:58:00 -0000 [thread overview]
Message-ID: <20110118191756.GA5032@bromo.med.uc.edu> (raw)
In-Reply-To: <AANLkTim3+GPb3GZ00gwVXi3q9A1R_HK=h7WfYCbagCQb@mail.gmail.com>
On Tue, Jan 18, 2011 at 12:34:19PM -0600, Sebastian Pop wrote:
> On Tue, Jan 18, 2011 at 11:55, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> > On Tue, Jan 18, 2011 at 11:01:56AM -0600, Sebastian Pop wrote:
> >> On Mon, Jan 17, 2011 at 23:37, Sebastian Pop <sebpop@gmail.com> wrote:
> >> > I will commit these patches to the graphite branch for further tests
> >> > sometime tomorrow if there are no further comments.
> >>
> >> Committed to the graphite branch r168953 - 168957.
> >>
> >> Sebastian
> >
> > Sebastian,
> > When I try to build with these patches applied to gcc trunk, the build fails at...
> >
> > gcc -c -g -fkeep-inline-functions -DIN_GCC -W -Wall -Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -Wold-style-definition -Wc++-compat -fno-common -DHAVE_CONFIG_H -I. -I. -I../../gcc-4.6-20110118/gcc -I../../gcc-4.6-20110118/gcc/. -I../../gcc-4.6-20110118/gcc/../include -I../../gcc-4.6-20110118/gcc/../libcpp/include -I/sw/include -I/sw/include -I../../gcc-4.6-20110118/gcc/../libdecnumber -I../../gcc-4.6-20110118/gcc/../libdecnumber/dpd -I../libdecnumber -I/sw/include -I/sw/include -DCLOOG_INT_GMP -DCLOOG_ORG -I/sw/include ../../gcc-4.6-20110118/gcc/omega.c -o omega.o
> > ../../gcc-4.6-20110118/gcc/omega.c:188: error: redefinition of 'gcd'
> > ../../gcc-4.6-20110118/gcc/hwint.h:236: error: previous definition of 'gcd' was here
> > make[3]: *** [omega.o] Error 1
> > make[2]: *** [all-stage1-gcc] Error 2
> > make[1]: *** [stage1-bubble] Error 2
> > make: *** [all] Error 2
> >
> > It appears there is yet another definition of gcd in omega.c.
>
> Yes, I corrected all this on the patch that I committed to the graphite branch.
> You can either get it from there, or I can send it to you.
>
> Thanks for your testing,
> Sebastian
Sebastian,
No problem. I removed the redefinition of 'gcd' from gcc/omega.c by hand and
the resulting build on x86_64-apple-darwin10 with...
Configured with: ../gcc-4.6-20110118/configure --prefix=/sw --prefix=/sw/lib/gcc4.6 --mandir=/sw/share/man --infodir=/sw/lib/gcc4.6/info --with-build-config=bootstrap-lto --enable-stage1-languages=c,lto --enable-languages=c,c++,fortran,lto,objc,obj-c++,java --with-gmp=/sw --with-libiconv-prefix=/sw --with-ppl=/sw --with-cloog=/sw --with-mpc=/sw --with-system-zlib --x-includes=/usr/X11R6/include --x-libraries=/usr/X11R6/lib --program-suffix=-fsf-4.6 --enable-checking=yes --enable-cloog-backend=isl
passes 'make -k check RUNTESTFLAGS="graphite.exp --target_board=unix'{-m32,-m64}'"' without regressions.
I'll do a full regression test next.
Jack
prev parent reply other threads:[~2011-01-18 19:18 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-18 5:38 Sebastian Pop
2011-01-18 5:38 ` [PATCH 1/5] Relax the definition of same_pdr_p Sebastian Pop
2011-01-18 5:40 ` [PATCH 3/5] Allow several reductions in a reduction PBB Sebastian Pop
2011-01-18 7:45 ` [PATCH 2/5] Remove duplicate close phi nodes in the canonical close phi representation Sebastian Pop
2011-01-18 8:45 ` [PATCH 4/5] Do not create the temporary array for reductions into VAR_DECLs Sebastian Pop
2011-01-18 10:50 ` Richard Guenther
2011-01-18 9:03 ` [PATCH 5/5] Remove the lambda framework and make -ftree-loop-linear an alias of -floop-interchange Sebastian Pop
2011-01-18 10:52 ` Richard Guenther
2011-01-18 16:48 ` Sebastian Pop
2011-01-25 23:39 ` Joseph S. Myers
2011-01-27 10:38 ` Jay Foad
2011-01-28 18:12 ` [PATCH] Use HOST_WIDE_INTs in gcd and least_common_multiple Sebastian Pop
2011-01-28 19:01 ` Joseph S. Myers
2011-01-28 22:18 ` Sebastian Pop
2011-01-31 12:35 ` Richard Guenther
2011-01-31 22:37 ` Joseph S. Myers
2011-02-04 4:08 ` Sebastian Pop
2011-02-04 23:13 ` Richard Guenther
2011-07-29 8:27 ` Sebastian Pop
2011-07-30 7:22 ` Richard Henderson
2011-01-31 14:36 ` Jay Foad
2011-01-31 14:43 ` Joseph S. Myers
2011-01-31 14:53 ` Jay Foad
2011-01-26 12:22 ` [PATCH 5/5] Remove the lambda framework and make -ftree-loop-linear an alias of -floop-interchange Eric Botcazou
2011-01-26 13:32 ` Richard Guenther
2011-01-26 15:59 ` Eric Botcazou
2011-01-26 22:43 ` Eric Botcazou
2011-01-25 23:28 ` Joseph S. Myers
2011-01-28 22:42 ` [PATCH] Use Alias in common.opt to " Sebastian Pop
2011-01-29 9:58 ` Jack Howarth
2011-01-29 10:54 ` Sebastian Pop
2011-01-31 12:41 ` Richard Guenther
2011-01-18 14:46 ` [PATCH 0/5] Remove the LAMBDA framework Jack Howarth
2011-01-18 16:56 ` Sebastian Pop
2011-01-18 17:08 ` Sebastian Pop
2011-01-18 18:20 ` Jack Howarth
2011-01-18 18:54 ` Sebastian Pop
2011-01-18 19:58 ` Jack Howarth [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110118191756.GA5032@bromo.med.uc.edu \
--to=howarth@bromo.med.uc.edu \
--cc=gcc-graphite@googlegroups.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=rguenther@suse.de \
--cc=sebpop@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).