OSDN Git Service

2008-08-18 Richard Guenther <rguenther@suse.de>
authorrguenth <rguenth@138bc75d-0d04-0410-961f-82ee72b054a4>
Mon, 18 Aug 2008 15:36:15 +0000 (15:36 +0000)
committerrguenth <rguenth@138bc75d-0d04-0410-961f-82ee72b054a4>
Mon, 18 Aug 2008 15:36:15 +0000 (15:36 +0000)
* tree-ssa-reassoc.c (reassociate_bb): Properly reset the
statement iterator after statement removal.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@139201 138bc75d-0d04-0410-961f-82ee72b054a4

gcc/ChangeLog
gcc/tree-ssa-reassoc.c

index 2f49559..38d137f 100644 (file)
@@ -1,3 +1,8 @@
+2008-08-18  Richard Guenther  <rguenther@suse.de>
+
+       * tree-ssa-reassoc.c (reassociate_bb): Properly reset the
+       statement iterator after statement removal.
+
 2008-08-18  Andreas Tobler  <a.tobler@schweiz.org>
 
        * config/rs6000/driver-rs6000.c (detect_caches_freebsd): New function.
index be68331..e4e7db6 100644 (file)
@@ -1771,6 +1771,18 @@ reassociate_bb (basic_block bb)
                {
                  gsi_remove (&gsi, true);
                  release_defs (stmt);
+                 /* We might end up removing the last stmt above which
+                    places the iterator to the end of the sequence.
+                    Reset it to the last stmt in this case which might
+                    be the end of the sequence as well if we removed
+                    the last statement of the sequence.  In which case
+                    we need to bail out.  */
+                 if (gsi_end_p (gsi))
+                   {
+                     gsi = gsi_last_bb (bb);
+                     if (gsi_end_p (gsi))
+                       break;
+                   }
                }
              continue;
            }