Fix for PR5373, Credit to Jakub Staszak.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@89758 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Transforms/Scalar/LoopUnswitch.cpp b/lib/Transforms/Scalar/LoopUnswitch.cpp
index 38d267aa6b8d7aafe723afa9e11675b357e6d0c6..382e2612cbd0be7d2a64651cbbd64a33d9327dee 100644 (file)
--- a/lib/Transforms/Scalar/LoopUnswitch.cpp
+++ b/lib/Transforms/Scalar/LoopUnswitch.cpp
@@ -296,7 +296,6 @@ static bool isTrivialLoopExitBlockHelper(Loop *L, BasicBlock *BB,
     // first exit.
     if (ExitBB != 0) return false;
     ExitBB = BB;
-    return true;
   }
   
   // Otherwise, this is an unvisited intra-loop node.  Check all successors.

diff --git a/test/Transforms/LoopUnswitch/5373.ll b/test/Transforms/LoopUnswitch/5373.ll
new file mode 100644 (file)
index 0000000..77b7835
--- /dev/null
+++ b/test/Transforms/LoopUnswitch/5373.ll
@@ -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