df housekeeping patch

Andreas Jaeger aj@suse.de
Sun Jan 26 09:38:00 GMT 2003


Michael Hayes <m.hayes@elec.canterbury.ac.nz> writes:

>  
> I've commited the following housekeeping patch for df.[ch].  It
> tidies up the formatting, clarifies comments, and removes dead code.
> It does not change any functionality.
>
> 2003-01-26  Michael Hayes  <mph@paradise.net.nz>
>
> 	* df.h: Update comments, tidy formatting.
> 	(DF_FORWARD, DF_REVERSE, DF_UNION, DF_INTERSECTION): Rename from FORWARD,
> 	REVERSE, UNION, INTERSECTION.  All uses updated.
> 	(OLD_DF_INTERFACE): Remove.
> 	(struct insn_info): Remove commented out insn field.
> 	* df.c: Update comments, tidy formatting.
> 	(df_def_table_realloc): Remove.

This patch was not tested at all since the tree does not bootstrap
with it.  You introduce a new, unused declaration:

> [...]
> *************** static struct ref *df_bb_insn_regno_firs
> *** 276,281 ****
> --- 290,296 ----
>   static void df_chain_dump PARAMS((struct df_link *, FILE *file));
>   static void df_chain_dump_regno PARAMS((struct df_link *, FILE *file));
>   static void df_regno_debug PARAMS ((struct df *, unsigned int, FILE *));
> + static void df_regno_rtl_debug PARAMS ((struct df *, unsigned int, FILE *));
>   static void df_ref_debug PARAMS ((struct df *, struct ref *, FILE *));
>   static void df_rd_transfer_function PARAMS ((int, int *, bitmap, bitmap,
>   					     bitmap, bitmap, void *));

I've removed that prototype again.  Michael, can you please test even
such tidying patches and check for warnings?

Andreas
-- 
 Andreas Jaeger
  SuSE Labs aj@suse.de
   private aj@arthur.inka.de
    http://www.suse.de/~aj



More information about the Gcc-patches mailing list