Re: [tree-ssa] Bootstrap broken on PPC

[Jan Hubicka <jh at suse dot cz>]

> 
> Jan,
> 
> This breakage appears related to your changes.
> 
> ----------------------------------------------------------------------------
> /home/cygnus/dnovillo/perf/sbox/tree-ssa-branch/local.ppc64/src/gcc/builtins.c:5941: warning: 'result' might be used uninitialized in this function

It is not in functio I've touched.  It looks like that one of our
optimizations confuse CFG enought so there is new path to exit not
initializing result (the result is initialized in many interesting ways
inside switch statement).
I would suspect that this is result of combination of fact that edge
splitting is broken (it redirects CFG edges without updating  the
underlying program code and it also invalidate SSA form by not updating
PHI nodes for new edges as noticed by verify_flow_info patch I sent)
and Daniel's changes to make ssapre more aggressive.
I will investigate this at evening, but I don't have access to PPC right
now so not sure how far I can get.
Can I have preprocessed builtins.c?

Honzaa
> make[2]: *** [builtins.o] Error 1
> make[2]: Leaving directory `/mnt/build/dnovillo/sbox/tree-ssa-branch.ppc64/bld.cure81/gcc'
> make[1]: *** [stage2_build] Error 2
> make[1]: Leaving directory `/mnt/build/dnovillo/sbox/tree-ssa-branch.ppc64/bld.cure81/gcc'
> make: *** [bootstrap] Error 2
> ----------------------------------------------------------------------------
> 
> --- gcc.prev/gcc/ChangeLog.tree-ssa	2003-11-05 11:28:05.000000000 -0500
> +++ gcc/gcc/ChangeLog.tree-ssa	2003-11-07 08:02:06.000000000 -0500
> @@ -1,3 +1,58 @@
> +2003-11-07  Jan Hubicka  <jh@suse.cz>
> +
> +	* fold-const.c (tree_expr_nonzero_p): Fix typo.
> +
> +	* fold-const.c (tree_expr_nonzero_p): New function.
> +	(fold_relational_const): Use it.
> +	(nondestructive_fold_binary_to_constant): Allow casts in address
> +	expressions.
> +
> +2003-11-06  Jan Hubicka  <jh@suse.cz>
> +
> +	* tree-cfg.c (tree_block_label):  Cleanup.
> +
> +2003-11-06  Daniel Berlin  <dberlin@dberlin.org>
> +
> +	* tree-ssa-pre.c (fast_a_dominates_b): New function.
> +	(build_dfs_id_array_1): Ditto.
> +	(build_dfs_id_array): Ditto.
> +	(load_modified_phi_result): Use fast_a_dominates_b.
> +	(rename_1): Ditto.
> +	Also use build_dfs_id_array, and remove some duplicate ephi_at_block
> +	calls.
> +	(insert_occ_in_preorder_dt_order): Remove some duplicate ephi_at_block
> +	calls.
> +	(pre_expression): Ditto. 
> +	Also free dfs_id arrays here.
> +	(collect_expressions): Remove duplicate bsi_stmt calls.
> +
> +2003-11-06  Daniel Berlin  <dberlin@dberlin.org>
> +	
> +	* tree-ssa-pre.c (count_stmts_in_bb): Removed.
> +	(set_var_phis): Only call bb_for_stmt once.
> +	(insert_one_operand): Remove endtree, endtreep, a lot of special handling
> +	no longer needed.  Remove insert_done.
> +	(collect_expressions): Enable INDIRECT_REF and SSA_NAME handling.
> +	
> +2003-11-06  Steven Bosscher  <stevenb@suse.de>
> +
> +	* tree-cfg.c (STRIP_CONTAINERS): Remove.
> +
> +2003-11-06  Jan Hubicka  <jh@suse.cz>
> +
> +	* tree-cfg.c (cleanup_cond_expr_graph):  Clean edge flags.
> +
> +2003-11-06  Zdenek Dvorak  <rakdver@atrey.karlin.mff.cuni.cz>
> +
> +	* tree-dump.c (dump_options): Remove TDF_LINENO from all setting.
> +
> +2003-11-06  Jan Hubicka  <jh@suse.cz>
> +
> +	* builtins.c (expand_builtin_strstr, expand_builtin_strchr,
> +	expand_builtin_strrchr, expand_builtin_strpbrk,
> +	simplify_builtin_strstr, simplify_builtin_strrchr,
> +	simplify_builtin_strpbrk): Add missing casts.
> +
> 
> 
> 
> Thanks.  Diego.
>