Easwaran Raman <era...@google.com> writes: > In cfgexpand.c, variables in non-overlapping lexical scopes are > assigned same stack locations at -O1 and above. At -O0, this is > attempted only if the size of the stack objects is above a threshold > (32). The rationale is at -O0, more variables are going to be in the > stack and the O(n^2) stack slot sharing algorithm will increase the > compilation time. This patch replaces the constant with a param which > is set to 32 by default. We ran into a case where the presence of > always_inline attribute triggered Wframe-larger-than warnings at -O0 > but not at -O2 since the different inlined copies share the stack. We > are ok with a slight increase in compilation time to get smaller stack > frames even at -O0 and this patch would allow us do that easily.
Seems like a odd thing for a param. If the compile time increase is very small (< 1%?) I would just make the new threshold default. -Andi -- a...@linux.intel.com -- Speaking for myself only
