[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