Re: [patch] Remove expand_expr_stmt

[Richard Guenther]

On Sat, Jul 14, 2012 at 7:34 PM, Steven Bosscher <stevenb....@gmail.com> wrote:
> Hello,
>
> This patch removes expand_expr_stmt and inlines+folds it into its only user.
>
> This relies on TARGET_STACK_PROTECT_FAIL being a CALL_EXPR. The
> documentation only says "a tree expression that (...) should involve a
> call to a @code{noreturn} function". I've changed this to explicitly
> state that the hook must return a CALL_EXPR.
>
> Bootstrapped and tested on powerpc64-unknown-linux-gnu.
> OK for trunk?

Ok.

Thanks,
Richard.

> Ciao!
> Steven
>
>
>         * function.c (stack_protect_epilogue): Use expand_call to expand
>         targetm.stack_protect_fail.
>         * stmt.c (expand_expr_stmt): Remove now-unused function.
>         * tree.h (expand_expr_stmt): Remove prototype.
>         * doc/tm.texi.in (TARGET_STACK_PROTECT_FAIL): Document that this
>         hook must return a CALL_EXPR.
>         * doc/tm.texi: Regenerate.