Re: VEC re-write [patch 01/25]

2012-11-25 Thread Richard Biener
On Sun, Nov 18, 2012 at 6:12 PM, Diego Novillo dnovi...@google.com wrote: On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that

Re: VEC re-write [patch 01/25]

2012-11-25 Thread Diego Novillo
On Sun, Nov 25, 2012 at 8:11 AM, Richard Biener richard.guent...@gmail.com wrote: Doesn't that mean that a non-GCC host compiler might fail to build 4.8 during stage1? Yes. Fixed last week. vec needed to become a pure POD. So sad. Diego.

Re: VEC re-write [patch 01/25]

2012-11-20 Thread Ulrich Weigand
Diego Novillo wrote: On Thu, Nov 15, 2012 at 4:53 PM, Diego Novillo dnovi...@google.com wrote: I have tested the patch pretty extensively: - Regular bootstraps on x86_64, ppc, ia64, sparc and hppa. - Bootstraps with --enable-checking=release - Bootstraps with --enable-checking=gc,gcac

Re: VEC re-write [patch 01/25]

2012-11-19 Thread David Edelsohn
On Sun, Nov 18, 2012 at 10:18 PM, Ian Lance Taylor i...@google.com wrote: On Sun, Nov 18, 2012 at 5:14 PM, David Edelsohn dje@gmail.com wrote: The problem is AIX stdlib.h defines #define vec_free free Ouch. I am not sure where #undef vec_free should be placed. In vec.h or

Re: VEC re-write [patch 01/25]

2012-11-19 Thread Jack Howarth
On Sun, Nov 18, 2012 at 11:15:22AM -0800, Andrew Pinski wrote: On Sun, Nov 18, 2012 at 9:12 AM, Diego Novillo dnovi...@google.com wrote: On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25

Re: VEC re-write [patch 01/25]

2012-11-19 Thread Jack Howarth
On Mon, Nov 19, 2012 at 02:57:08PM -0500, Jack Howarth wrote: On Sun, Nov 18, 2012 at 11:15:22AM -0800, Andrew Pinski wrote: On Sun, Nov 18, 2012 at 9:12 AM, Diego Novillo dnovi...@google.com wrote: On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat,

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Hans-Peter Nilsson
On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that you think may be related to this rewrite. That seems to have trigged some bug in gcc-4.4-era. See PR55381. There are a lot of suspicious warnings from

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Diego Novillo
On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that you think may be related to this rewrite. That seems to have trigged some

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Hans-Peter Nilsson
On Sun, 18 Nov 2012, Diego Novillo wrote: My cris-elf builds worked fine, but config-list.mk only builds stage 1, it does not build libgfortran. Can you give me instructions on how to build your target on my x86 workstation? Better see the PR for cc1 command-line and preprocessed C-file.

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Diego Novillo
On Sun, Nov 18, 2012 at 12:43 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sun, 18 Nov 2012, Diego Novillo wrote: My cris-elf builds worked fine, but config-list.mk only builds stage 1, it does not build libgfortran. Can you give me instructions on how to build your target on my x86

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Andreas Tobler
On 18.11.12 18:05, Hans-Peter Nilsson wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that you think may be related to this rewrite. That seems to have trigged some bug in gcc-4.4-era. See

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Hans-Peter Nilsson
On Sun, 18 Nov 2012, Andreas Tobler wrote: Is there a minimum gcc to bootstrap current trunk? I currently fail to bootstrap trunk with a 4.2.1 gcc, but a 4.6 succeeds. A gcc-4.1.2 has worked for me in the past, before this recent vec.h change. I think that's the minimum version reportedly

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Andrew Pinski
On Sun, Nov 18, 2012 at 9:12 AM, Diego Novillo dnovi...@google.com wrote: On Sun, Nov 18, 2012 at 12:05 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Sat, 17 Nov 2012, Diego Novillo wrote: I have now committed all 25 parts of this patch as rev 193595. Please CC me on any problems that

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Andreas Tobler
On 18.11.12 20:11, Hans-Peter Nilsson wrote: On Sun, 18 Nov 2012, Andreas Tobler wrote: Is there a minimum gcc to bootstrap current trunk? I currently fail to bootstrap trunk with a 4.2.1 gcc, but a 4.6 succeeds. A gcc-4.1.2 has worked for me in the past, before this recent vec.h change.

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Hans-Peter Nilsson
On Sun, 18 Nov 2012, Andreas Tobler wrote: On 18.11.12 20:11, Hans-Peter Nilsson wrote: On Sun, 18 Nov 2012, Andreas Tobler wrote: Is there a minimum gcc to bootstrap current trunk? I currently fail to bootstrap trunk with a 4.2.1 gcc, but a 4.6 succeeds. A gcc-4.1.2 has worked for

Re: VEC re-write [patch 01/25]

2012-11-18 Thread David Edelsohn
Files were changed in gcc/c-family with no associated ChangeLog entry. And now bootstrap fails on AIX using GCC 4.6.3 with the error: /nasfarm/dje/src/src/gcc/c-family/c-lex.c: In function 'c_fileinfo* get_fileinfo(const char*)': /nasfarm/dje/src/src/gcc/c-family/c-lex.c:107:39: error:

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Diego Novillo
On Sun, Nov 18, 2012 at 4:28 PM, David Edelsohn dje@gmail.com wrote: Files were changed in gcc/c-family with no associated ChangeLog entry. *gasp* I mistakenly put them in c/ChangeLog. Not that they carry any useful information, but I'll move them. And now bootstrap fails on AIX using

Re: VEC re-write [patch 01/25]

2012-11-18 Thread David Edelsohn
On Sun, Nov 18, 2012 at 8:03 PM, Diego Novillo dnovi...@google.com wrote: And now bootstrap fails on AIX using GCC 4.6.3 with the error: /nasfarm/dje/src/src/gcc/c-family/c-lex.c: In function 'c_fileinfo* get_fileinfo(const char*)': /nasfarm/dje/src/src/gcc/c-family/c-lex.c:107:39: error:

Re: VEC re-write [patch 01/25]

2012-11-18 Thread Ian Lance Taylor
On Sun, Nov 18, 2012 at 5:14 PM, David Edelsohn dje@gmail.com wrote: The problem is AIX stdlib.h defines #define vec_free free Ouch. I am not sure where #undef vec_free should be placed. In vec.h or system.h? I think system.h. Ian

Re: VEC re-write [patch 01/25]

2012-11-17 Thread Diego Novillo
On Thu, Nov 15, 2012 at 4:53 PM, Diego Novillo dnovi...@google.com wrote: I have tested the patch pretty extensively: - Regular bootstraps on x86_64, ppc, ia64, sparc and hppa. - Bootstraps with --enable-checking=release - Bootstraps with --enable-checking=gc,gcac - Basic builds on all

Re: VEC re-write [patch 01/25]

2012-11-16 Thread Diego Novillo
On Fri, Nov 16, 2012 at 5:23 AM, Pedro Alves pal...@redhat.com wrote: Was this considered? Yup. I just did not implement it. Would be a good follow up, though. Diego.

Re: VEC re-write [patch 01/25]

2012-11-16 Thread Pedro Alves
On 16-11-2012 12:13, Diego Novillo wrote: On Fri, Nov 16, 2012 at 5:23 AM, Pedro Alves pal...@redhat.com wrote: Was this considered? Yup. I just did not implement it. Would be a good follow up, though. Ah, good to know. Thanks. For the archives, cause gcc-patches@ bounced my mail for

Re: VEC re-write [patch 01/25]

2012-11-16 Thread Ian Lance Taylor
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo dnovi...@google.com wrote: 2012-11-15 Diego Novillo dnovi...@google.com * vec.c (register_overhead): Convert it into member function of vec_prefix. (release_overhead): Likewise. (calculate_allocation):

VEC re-write [patch 01/25]

2012-11-15 Thread Diego Novillo
I have split the VEC rewrite into 25 patches. The only patches that make actual changes are #1 (vec.c and vec.h) and #2 (gengtype). All the others are mechanical side-effects from the first patch. I will still appreciate if maintainers can take a look at the other patches to make sure I have