[tuples][patch] Remove gimple_seq_has_side_effects, fix gimplifier to correctly wrap the function body into try/finally block, if required

Diego Novillo dnovillo@google.com
Tue Mar 25 15:22:00 GMT 2008


On 3/24/08 8:49 PM, Oleg Ryjkov wrote:

> @@ -1157,6 +1158,7 @@ gimplify_bind_expr (tree *expr_p, gimple
>    body = NULL;
>    gimplify_stmt (&BIND_EXPR_BODY (bind_expr), &body);
>    gimple_bind_set_body (gimple_bind, body);
> +  gimple_set_block (gimple_bind, BIND_EXPR_BLOCK (bind_expr));

No need to set the block again.  We are now setting it in the call to 
gimple_build_bind.


> @@ -6865,12 +6865,15 @@ gimplify_function_tree (tree fndecl)
>        x = implicit_built_in_decls[BUILT_IN_PROFILE_FUNC_ENTER];
>        gimplify_seq_add_stmt (&body, gimple_build_call (x, 0));
>        gimplify_seq_add_stmt (&body, tf);
> -      bind = gimple_build_bind (NULL, body);
> +      new_bind = gimple_build_bind (NULL, body, gimple_block (bind));
> +      /* Clear the block of bind, since it is no longer directly inside

s/of bind/for BIND/

OK with those changes.


Diego.



More information about the Gcc-patches mailing list