On Thu, 20 Feb 2003, Simon Glover wrote: > > On Thu, 20 Feb 2003, Nicholas Clark wrote: > > > If I > > > > perl Configure.pl --cgoto=0 && make all test > > > > then the build fails with: > > > > ccache /usr/local/bin/gcc -DHAS_FPSETMASK -DHAS_FLOATINGPOINT_H > > -I/usr/local/include -Wall -Wstrict-prototypes -Wmissing-prototypes -Winline > > -Wshadow -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-strings > > -Waggregate-return -Winline -W -Wno-unused -Wsign-compare -Wformat-nonliteral > > -Wformat-security -Wpacked -Wpadded -Wdisabled-optimization -I./include > > -DHAS_JIT -DI386 -o jit_cpu.o -c jit_cpu.c > > In file included from jit_cpu.c:39: > > include/parrot/jit_emit.h:2302:39: parrot/oplib/core_ops_cgp.h: No such file or > > directory > > In file included from jit_cpu.c:39: > > include/parrot/jit_emit.h: In function `Parrot_jit_begin': > > include/parrot/jit_emit.h:2349: `cgp_core' undeclared (first use in this function) > > include/parrot/jit_emit.h:2349: (Each undeclared identifier is reported only once > > include/parrot/jit_emit.h:2349: for each function it appears in.) > > *** Error code 1 > > The problem is that an ifdef in jit/i386/jit_emit.h is defining JIT_CGP > based on whether or not the compiler is GCC, and not on whether > HAS_COMPUTED_GOTO is defined. The attached patch fixes this, but I'm > not sure if the __GCC__ bit is still necessary. Leo?
OK, let's try this again, with the _correct_ spelling this time... Simon --- jit/i386/jit_emit.h.old Thu Feb 20 21:43:53 2003 +++ jit/i386/jit_emit.h Thu Feb 20 21:43:20 2003 @@ -8,7 +8,7 @@ #include <assert.h> -#ifdef __GNUC__ +#if defined HAVE_COMPUTED_GOTO && defined __GCC__ # define JIT_CGP #endif