This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Obvious cleanup


Hi,
I've installed the attached patch as obvious.  There is no longer reason
for having that function as expunging blocks is cheap.
Thu Jun 12 19:59:34 CEST 2003  Jan Hubicka  <jh@suse.cz>
	* basic-block.h (flow_delete_block_noexpunge): Kill.
	* cfgrtl.c (flow_delete_block_noexpunge): Merge to
	rtl_delete_block.
Index: basic-block.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/basic-block.h,v
retrieving revision 1.174
diff -c -3 -p -r1.174 basic-block.h
*** basic-block.h	7 Jun 2003 21:30:47 -0000	1.174
--- basic-block.h	12 Jun 2003 17:57:05 -0000
*************** extern edge redirect_edge_succ_nodup	PAR
*** 362,368 ****
  extern void redirect_edge_pred		PARAMS ((edge, basic_block));
  extern basic_block create_basic_block_structure PARAMS ((rtx, rtx, rtx, basic_block));
  extern basic_block create_basic_block	PARAMS ((rtx, rtx, basic_block));
- extern void flow_delete_block_noexpunge	PARAMS ((basic_block));
  extern void clear_bb_flags		PARAMS ((void));
  extern void merge_blocks_nomove		PARAMS ((basic_block, basic_block));
  extern void tidy_fallthru_edge		PARAMS ((edge, basic_block,
--- 362,367 ----
Index: cfgrtl.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/cfgrtl.c,v
retrieving revision 1.79
diff -c -3 -p -r1.79 cfgrtl.c
*** cfgrtl.c	8 Jun 2003 14:50:10 -0000	1.79
--- cfgrtl.c	12 Jun 2003 17:57:06 -0000
*************** create_basic_block (head, end, after)
*** 357,364 ****
  /* ??? Preserving all such notes strikes me as wrong.  It would be nice
     to post-process the stream to remove empty blocks, loops, ranges, etc.  */
  
! void
! flow_delete_block_noexpunge (b)
       basic_block b;
  {
    rtx insn, end, tmp;
--- 357,364 ----
  /* ??? Preserving all such notes strikes me as wrong.  It would be nice
     to post-process the stream to remove empty blocks, loops, ranges, etc.  */
  
! static void
! rtl_delete_block (b)
       basic_block b;
  {
    rtx insn, end, tmp;
*************** flow_delete_block_noexpunge (b)
*** 412,424 ****
  
    b->pred = NULL;
    b->succ = NULL;
- }
- 
- static void
- rtl_delete_block (b)
-      basic_block b;
- {
-   flow_delete_block_noexpunge (b);
  
    /* Remove the basic block from the array.  */
    expunge_block (b);
--- 412,417 ----


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]