[llvm] r365600 - [SimpleLoopUnswitch] Add a test case exposing a bug

Serguei Katkov via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 10 01:25:48 PDT 2019


Author: skatkov
Date: Wed Jul 10 01:25:48 2019
New Revision: 365600

URL: http://llvm.org/viewvc/llvm-project?rev=365600&view=rev
Log:
[SimpleLoopUnswitch] Add a test case exposing a bug

This test exposes a bug in SimpleLoopUnswitch that leads to a crash on
assert(SuccessorsCount > 1 && "Cannot unswitch a condition without multiple distinct successors!");
when SimpleLoopUnswitch considers unswitching of a loop by a switch with one successor.

Fix will be submitted soon.

Patch Author: Daniil Suchkov.

Reviewers: reames, asbirlea, skatkov
Reviewed By: skatkov
Subscribers: zzheng, llvm-commits
Differential Revision: https://reviews.llvm.org/D64403

Added:
    llvm/trunk/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-redundant-switch.ll

Added: llvm/trunk/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-redundant-switch.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-redundant-switch.ll?rev=365600&view=auto
==============================================================================
--- llvm/trunk/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-redundant-switch.ll (added)
+++ llvm/trunk/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-redundant-switch.ll Wed Jul 10 01:25:48 2019
@@ -0,0 +1,25 @@
+; REQUIRES: asserts
+; XFAIL: *
+; RUN: opt -passes='unswitch<nontrivial>' -disable-output -S < %s
+; RUN: opt -simple-loop-unswitch -enable-nontrivial-unswitch -disable-output -S < %s
+
+; This loop shouldn't trigger asserts in SimpleLoopUnswitch.
+define void @test_redundant_switch(i1* %ptr, i32 %cond) {
+entry:
+  br label %loop_begin
+
+loop_begin:
+  switch i32 %cond, label %loop_body [
+      i32 0, label %loop_body
+  ]
+
+loop_body:
+  br label %loop_latch
+
+loop_latch:
+  %v = load i1, i1* %ptr
+  br i1 %v, label %loop_begin, label %loop_exit
+
+loop_exit:
+  ret void
+}




More information about the llvm-commits mailing list