[tuples] Enable more calls cleanup_tree_cfg

[tuples] Enable more calls cleanup_tree_cfg

[Diego Novillo]

This enables pass_cleanup_cfg.  Tested on x86_64.  Committed.

Diego.

Re: [tuples] Enable more calls cleanup_tree_cfg

[Aldy Hernandez]

>>>>> "Diego" == Diego Novillo <dnovillo@google.com> writes:

 > This enables pass_cleanup_cfg.  Tested on x86_64.  Committed.
 > Diego.

Where's the patch?

Re: [tuples] Enable more calls cleanup_tree_cfg

[Diego Novillo]

[-- Attachment #1: Type: text/plain, Size: 130 bytes --]

On 11/1/07, Aldy Hernandez <aldyh@redhat.com> wrote:

> Where's the patch?

2 for 2.  I'm on a roll, sorry.  Here it is.


Diego.

[-- Attachment #2: 20071029-enable-cfg-cleanup.diff.txt --]
[-- Type: text/plain, Size: 1825 bytes --]

2007-10-29  Diego Novillo  <dnovillo@google.com>

	* tree-optimize.c (execute_cleanup_cfg_pre_ipa): Re-enable.
	* gimple.c (walk_gimple_asm): Tidy comment.

Index: tree-optimize.c
===================================================================
--- tree-optimize.c	(revision 129739)
+++ tree-optimize.c	(working copy)
@@ -150,13 +150,8 @@ struct tree_opt_pass pass_all_early_opti
 static unsigned int
 execute_cleanup_cfg_pre_ipa (void)
 {
-  /* FIXME tuples  */
-#if 0
   cleanup_tree_cfg ();
   return 0;
-#else
-  gcc_unreachable ();
-#endif
 }
 
 struct tree_opt_pass pass_cleanup_cfg =
@@ -174,7 +169,7 @@ struct tree_opt_pass pass_cleanup_cfg =
   0,					/* todo_flags_start */
   TODO_dump_func,					/* todo_flags_finish */
   0					/* letter */
-  ,0					/* works_with_tuples_p */
+  ,1					/* works_with_tuples_p */
 };
 
 
Index: gimple.c
===================================================================
--- gimple.c	(revision 129739)
+++ gimple.c	(working copy)
@@ -1009,11 +1009,12 @@ walk_gimple_asm (gimple stmt, walk_tree_
 
 
 
-/* Walk GIMPLE statement STMT using traversal state stored in WI.  The
-   callback CALLBACK_STMT is called.  If CALLBACK_STMT returns true,
-   it means that the callback function has handled all the operands of
-   STMT and it is no necessary to walk STMT's operands.
-   
+/* Walk GIMPLE statement STMT (optionally using traversal state stored
+   in WI).  If WI is NULL, no state is kept during traversal.
+   The callback CALLBACK_STMT is called.  If CALLBACK_STMT returns
+   true, it means that the callback function has handled all the
+   operands of STMT and it is no necessary to walk STMT's operands.
+
    Otherwise, CALLBACK_OP is called on each operand of STMT.  If
    CALLBACK_OP returns non-NULL for an operand, the remaining operands
    are not scanned.