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

Reply via email to