On Fri, Jan 27, 2012 at 01:04:41PM -0800, Ian Lance Taylor wrote: > This patch fixes the problem by introducing a new builtin function > __builtin_init_heap_trampoline. This is a middle-end change so it is > covered under my maintainership. I also believe it is quite safe. > However, given that we are in stage 4 I would like to ask the release > managers if this is OK to put in. It fixes a user-reported bug, but the > bug is technically not a regression because it has never actually worked > correctly. > > Bootstrapped and ran full testsuite on x86_64-unknown-linux-gnu. OK to > commit?
Looks good to me, except I wonder whether the builtin needs to be user accessible. If not, can't you throw some * or space into the name, so that the builtin is really internal to gcc? Jakub