On Wed, 2016-07-27 at 11:57 +1000, David Gibson wrote: > > -void helper_float_check_status(CPUPPCState *env) > > +static __attribute__((noinline)) void > do_float_check_status(CPUPPCState *env, > > + > uintptr_t raddr) > > Why do you need to force this to be non-inline?
I don't. It just that gcc inlines too much shit :-) In fact I had a patch at some point that made a whole pile of stuff in translate.c non-inline and that reduced the size of it by about 100K But yeah, I can drop it for now, it's irrelevant to the point of the patch. Cheers, ben.