Messages in this thread | | | Subject | Re: [PATCH] lib/vsprintf: add %pT format specifier | From | Tetsuo Handa <> | Date | Sat, 11 Jan 2014 11:30:22 +0900 |
| |
Joe Perches wrote: > On Sat, 2014-01-11 at 10:59 +0900, Tetsuo Handa wrote: > > I just added noinline_for_stack as with other functions does. > > But indeed, stack used by name[] is only 16 bytes but stack used by function > > arguments are larger than 16 bytes. We should remove noinline_for_stack ? > > My recollection is that certain gcc versions don't do > well with multiple inline functions that have stack > variables. > > Instead of collapsing the variables, gcc will > accumulate the stack depth of all the inlines > args instead of reusing the stack. >
I tried what commit cf3b429b "vsprintf.c: use noinline_for_stack" says using gcc 4.4.7, but it made no difference for stack usage. Thus, it seems harmless to keep noinline_for_stack keyword.
| |