This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
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))
- From: "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Gerald Pfeifer <gerald at pfeifer dot com>, Jeff Law <law at redhat dot com>, "gcc at gcc dot gnu dot org" <gcc at gcc dot gnu dot org>
- Date: Thu, 31 Oct 2013 13:32:19 +0000
- Subject: 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))
- Authentication-results: sourceware.org; auth=none
- References: <BF230D13CA30DD48930C31D4099330003A495A50 at FMSMSX101 dot amr dot corp dot intel dot com> <20131031074533 dot GM27813 at tucnak dot zalov dot cz>
> -----Original Message-----
> From: Jakub Jelinek [mailto:jakub@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 failure
> (was: [PATCH, committed] libcilkrts - Add check for availability of alloca.h (Bug
> Bootstrap/58918))
>
> On Thu, Oct 31, 2013 at 03:21:14AM +0000, 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 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] with i?86. That should work fine right?
> Jakub