[tree-ssa] Fix warnings for PPC bootstraps [patch]
law@redhat.com
law@redhat.com
Mon Feb 3 23:55:00 GMT 2003
In message <20030203232554.GA4414@tornado.toronto.redhat.com>, Diego Novillo wr
ites:
>Fixes warnings that prevent bootstrapping on PPC. We should do
>something in the gimplifier to avoid emitting these silly
>warnings. Jason, any suggestions?
>
>There's one change I needed to make to t-rs6000 that puzzles me.
>Even though the port defines ASM_OUTPUT_ALIGNED_BSS, varasm.c
>doesn't seem to call asm_output_aligned_bss, so we get a warning
>for an unused static function. Could one of the PPC folks give
>me a hand with this? I re-added -Wno-error to varasm.o for now.
>
>Bootstrap still in progress. Will commit afterwards.
>
>
>Diego.
>
> Warning fixes for bootstrapping on PPC.
>
> * config/rs6000/rs6000.md (modsi3): Initialize variable 'i'.
> (moddi3): Likewise.
> * config/rs6000/t-rs6000 (varasm.o-warn): Add -Wno-error.
> * config/rs6000/rs6000.c (print_operand): Initialize variable 'i'.
>
> * c-typeck.c (build_component_ref): Initialize variable 'ref'.
> * cfgrtl.c (create_basic_block_structure): Initialize variable 'bb'.
> * combine.c (simplify_if_then_else): Initialize variables 'i'
> and 'nzb'.
> (simplify_set): Initialize variable 'cc_use'.
> * fold-const.c (fold_range_test): Initialize variable 'tem'.
> * ifcvt.c (noce_try_store_flag_constants): Initialize variable 'tmp'.
> (noce_process_if_block): Initialize variable 'set_a'.
> * jump.c (follow_jumps): Initialize variable 'insn'.
> * reload1.c (do_output_reload): Initialize variable 'note'.
> (gen_reload): Initialize variable 'tem'.
> * rtlanal.c (get_jump_table_offset): Initialize variable 'table'.
>
> * objc/objc-act.c (generate_ivar_lists): Initialize variable 'chain'.
>
>cp/ChangeLog.tree-ssa
>
> Warning fixes for bootstrapping on PPC.
>
> * cp/pt.c (tsubst_copy_and_build): Initialize variable 'id'.
> * cp/decl2.c (build_expr_from_tree): Initialize variable 'id'.
>
>f/ChangeLog.tree-ssa
>
> Warning fixes for bootstrapping on PPC.
>
> * f/where.c (ffewhere_track): Initialize variable 'lo'.
>
>
>java/ChangeLog.tree-ssa
>
> Warning fixes for bootstrapping on PPC.
>
> * java/expr.c (java_stack_swap): Initialize variables
> 'type1' and 'type2'.
Rather than do this (which makes effectively a lot of gratutious changes
to the compiler and makes our merge job harder), I'd rather see us work
on the gimplifier to avoid the bogons. I can't stress how much I think
adding all these changes on the SSA branch is a mistake.
Jeff
More information about the Gcc-patches
mailing list