Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure

2013-10-31 Thread Gerald Pfeifer
cOn Thu, 31 Oct 2013, Iyer, Balaji V wrote: > It should include a 3 too. My bad sorry. I have fixed it. Attached, > please find a patch. It is committed as obvious. Thanks. Now the next failure mode, I'm afraid. http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58951 libtool: link: /scratch2/tmp/ge

RE: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure

2013-10-31 Thread Iyer, Balaji V
@gcc.gnu.org Subject: Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure Jakub Jelinek writes: > On Thu, Oct 31, 2013 at 01:32:19PM +, Iyer, Balaji V wrote: >> > It is not just about not including 3, but also the []s in >> > configure.ac were eaten by m4. I

Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure

2013-10-31 Thread Rainer Orth
Jakub Jelinek writes: > On Thu, Oct 31, 2013 at 01:32:19PM +, Iyer, Balaji V wrote: >> > It is not just about not including 3, but also the []s in configure.ac >> > were eaten by >> > m4. In any case, shouldn't you fix also config/generic? >> > >> >> I am in the process of fixing config/ge

Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure (was: [PATCH, committed] libcilkrts - Add check for availability of alloca.h (Bug Bootstrap/58918))

2013-10-31 Thread Jakub Jelinek
On Thu, Oct 31, 2013 at 01:32:19PM +, Iyer, Balaji V wrote: > > It is not just about not including 3, but also the []s in configure.ac were > > eaten by > > m4. In any case, shouldn't you fix also config/generic? > > > > I am in the process of fixing config/generic. I also replaced [456] wi

RE: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure (was: [PATCH, committed] libcilkrts - Add check for availability of alloca.h (Bug Bootstrap/58918))

2013-10-31 Thread Iyer, Balaji V
> -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Thursday, October 31, 2013 3:46 AM > To: Iyer, Balaji V > Cc: Gerald Pfeifer; Jeff Law; gcc@gcc.gnu.org > Subject: Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c > f

Re: [PATCH] RE: libcilkrts/runtime/config/generic/cilk-abi-vla.c failure (was: [PATCH, committed] libcilkrts - Add check for availability of alloca.h (Bug Bootstrap/58918))

2013-10-31 Thread Jakub Jelinek
On Thu, Oct 31, 2013 at 03:21:14AM +, Iyer, Balaji V wrote: > It is because of this line: > > i[456]86-*-*) > config_dir="x86" > ;; > > It should include a 3 too. My bad sorry. I have fixed it. Attached, please > find a patch. It is committed as obvious. It is not just about not i