[llvm-commits] [llvm] r89758 - in /llvm/trunk: lib/Transforms/Scalar/LoopUnswitch.cpp test/Transforms/LoopUnswitch/5373.ll

[Edward O'Callaghan]

Author: evocallaghan
Date: Tue Nov 24 05:51:52 2009
New Revision: 89758

URL: http://llvm.org/viewvc/llvm-project?rev=89758&view=rev
Log:
Fix for PR5373, Credit to Jakub Staszak.

Added:
    llvm/trunk/test/Transforms/LoopUnswitch/5373.ll
Modified:
    llvm/trunk/lib/Transforms/Scalar/LoopUnswitch.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/LoopUnswitch.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopUnswitch.cpp?rev=89758&r1=89757&r2=89758&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopUnswitch.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LoopUnswitch.cpp Tue Nov 24 05:51:52 2009
@@ -296,7 +296,6 @@
     // first exit.
     if (ExitBB != 0) return false;
     ExitBB = BB;
-    return true;
   }
   
   // Otherwise, this is an unvisited intra-loop node.  Check all successors.

Added: llvm/trunk/test/Transforms/LoopUnswitch/5373.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/LoopUnswitch/5373.ll?rev=89758&view=auto

==============================================================================
--- llvm/trunk/test/Transforms/LoopUnswitch/5373.ll (added)
+++ llvm/trunk/test/Transforms/LoopUnswitch/5373.ll Tue Nov 24 05:51:52 2009
@@ -0,0 +1,24 @@
+; RUN: opt < %s -loop-unswitch -stats -disable-output |& grep "3 loop-unswitch - Number of branches unswitched"
+
+define noalias i32* @func_16(i32** %p_18, i32* %p_20) noreturn nounwind ssp {
+entry:
+  %lnot = icmp eq i32** %p_18, null               ; <i1> [#uses=1]
+  %lnot6 = icmp eq i32* %p_20, null               ; <i1> [#uses=1]
+  br label %for.body
+
+for.body:                                         ; preds = %cond.end, %entry
+  br i1 %lnot, label %cond.end, label %cond.true
+
+cond.true:                                        ; preds = %for.body
+  tail call void @f()
+  unreachable
+
+cond.end:                                         ; preds = %for.body
+  br i1 %lnot6, label %for.body, label %cond.true10
+
+cond.true10:                                      ; preds = %cond.end
+  tail call void @f()
+  unreachable
+}
+
+declare void @f() noreturn