[EMAIL PROTECTED] wrote on 28/08/2007 17:31:08:
> On 8/28/07, Dave Korn <[EMAIL PROTECTED]> wrote: > > On 28 August 2007 15:10, Richard Guenther wrote: > > > > > > > > Or maybe on ppc/spu enum bitfields are signed and the following > > > > > > DECL_FUNCTION_CODE (decl) = -1; > > > gcc_assert (DECL_FUNCTION_CODE (decl) >= function_code); > > > > > > doesnt work? > > > > I saw what you did there, and thought it slightly peculiar. Wouldn't > > > > DECL_FUNCTION_CODE (decl) = function_code; > > gcc_assert (DECL_FUNCTION_CODE (decl) == function_code); > > > > have worked just as well, or is there some gotcha you were trying to avoid with > > the dead store trick? > > Oh, indeed - this looks like it should have less problems. Revital, if that > works for you then changing the code above to what Dave suggests is also > pre-approved (but it really might be that SPU needs more than 1024 builtin > functions of a kind). It seems that replacing the above code does not solve the problem. Thanks, Revital
