Skip to content

Commit 1220c03

Browse files
committed
Fix test SILOpt test cases for edge splitting.
1 parent 4106b76 commit 1220c03

18 files changed

+566
-434
lines changed

test/SILOptimizer/abcopts.sil

Lines changed: 12 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -609,17 +609,15 @@ bb2:
609609

610610
// HOIST-LABEL: sil @hoist_rangechecked
611611
// HOIST: bb0
612-
// HOIST: cond_br {{.*}}, bb1{{.*}}, bb2
612+
// HOIST: cond_br {{.*}}, bb2, bb1
613613
// HOIST: bb1:
614-
// HOIST: br bb6
615-
// HOIST: bb2:
616614
// HOIST: [[CB:%[0-9]+]] = function_ref @checkbounds
617615
// HOIST: apply [[CB]]
618616
// HOIST: br bb3{{.*}}
619617
// HOIST: bb3{{.*}}:
620618
// HOIST-NOT: function_ref @checkbounds
621619
// HOIST-NOT: apply [[CB]]
622-
// HOIST: cond_br {{.*}}, bb5{{.*}}, bb4{{.*}}
620+
// HOIST: cond_br {{.*}}, bb5, bb4
623621
// HOIST: bb4
624622
// HOIST: br bb3
625623
// HOIST: bb5
@@ -671,17 +669,15 @@ bb3:
671669

672670
// HOIST-LABEL: sil @hoist_rangechecked_ref_tail_addr
673671
// HOIST: bb0
674-
// HOIST: cond_br {{.*}}, bb1{{.*}}, bb2
672+
// HOIST: cond_br {{.*}}, bb2, bb1
675673
// HOIST: bb1:
676-
// HOIST: br bb6
677-
// HOIST: bb2:
678674
// HOIST: [[CB:%[0-9]+]] = function_ref @checkbounds
679675
// HOIST: apply [[CB]]
680676
// HOIST: br bb3{{.*}}
681677
// HOIST: bb3{{.*}}:
682678
// HOIST-NOT: function_ref @checkbounds
683679
// HOIST-NOT: apply [[CB]]
684-
// HOIST: cond_br {{.*}}, bb5{{.*}}, bb4{{.*}}
680+
// HOIST: cond_br {{.*}}, bb5, bb4
685681
// HOIST: bb4
686682
// HOIST: br bb3
687683
// HOIST: bb5
@@ -818,17 +814,15 @@ bb2:
818814

819815
// HOIST-LABEL: sil @hoist_rangechecked_addr_proj_store
820816
// HOIST: bb0
821-
// HOIST: cond_br {{.*}}, bb1{{.*}}, bb2
817+
// HOIST: cond_br {{.*}}, bb2, bb1
822818
// HOIST: bb1:
823-
// HOIST: br bb6
824-
// HOIST: bb2:
825819
// HOIST: [[CB:%[0-9]+]] = function_ref @checkbounds
826820
// HOIST: apply [[CB]]
827821
// HOIST: br bb3{{.*}}
828822
// HOIST: bb3{{.*}}:
829823
// HOIST-NOT: function_ref @checkbounds
830824
// HOIST-NOT: apply [[CB]]
831-
// HOIST: cond_br {{.*}}, bb5{{.*}}, bb4{{.*}}
825+
// HOIST: cond_br {{.*}}, bb5, bb4
832826
// HOIST: bb4
833827
// HOIST: br bb3
834828
// HOIST: bb5
@@ -882,17 +876,15 @@ bb3:
882876

883877
// HOIST-LABEL: hoist_inclusive_rangechecked
884878
// HOIST: bb0
885-
// HOIST: cond_br {{.*}}, bb1{{.*}}, bb2
879+
// HOIST: cond_br {{.*}}, bb2, bb1
886880
// HOIST: bb1:
887-
// HOIST: br bb6
888-
// HOIST: bb2:
889881
// HOIST: [[CB:%[0-9]+]] = function_ref @checkbounds
890882
// HOIST: apply [[CB]]
891883
// HOIST: br bb3{{.*}}
892884
// HOIST: bb3{{.*}}:
893885
// HOIST-NOT: function_ref @checkbounds
894886
// HOIST-NOT: apply [[CB]]
895-
// HOIST: cond_br {{.*}}, bb5{{.*}}, bb4{{.*}}
887+
// HOIST: cond_br {{.*}}, bb5, bb4
896888
// HOIST: bb4:
897889
// HOIST: br bb3
898890
// HOIST: bb5:
@@ -944,16 +936,15 @@ bb3:
944936
/// Don't hoist arrays that are variant.
945937
// HOIST-LABEL: dont_hoist_variant_array
946938
// HOIST: bb0
947-
// HOIST: cond_br {{.*}}, bb1{{.*}}, bb2
939+
// HOIST: cond_br {{.*}}, bb2, bb1
948940
// HOIST: bb1:
949-
// HOIST: br bb6
941+
// HOIST-NEXT: br bb3
950942
// HOIST: bb2:
951-
// HOIST-NOT: function_ref @checkbounds
952-
// HOIST: br bb3{{.*}}
943+
// HOIST-NEXT: br bb6
953944
// HOIST: bb3{{.*}}:
954945
// HOIST: [[CB:%[0-9]+]] = function_ref @checkbounds
955946
// HOIST: apply [[CB]]
956-
// HOIST: cond_br {{.*}}, bb5{{.*}}, bb4{{.*}}
947+
// HOIST: cond_br {{.*}}, bb5, bb4
957948
// HOIST: bb4:
958949
// HOIST: br bb3
959950
// HOIST: bb5:

test/SILOptimizer/arcsequenceopts_knownsafebugs_loop.sil

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -202,10 +202,10 @@ bb2:
202202
// CHECK-LABEL: sil hidden @$unmatched_rr_loop_early_exit :
203203
// CHECK: bb1
204204
// CHECK: strong_release %0 : $Cls
205-
// CHECK: cond_br undef, bb3, bb2
206-
// CHECK: bb3:
205+
// CHECK: cond_br undef, bb2, bb3
206+
// CHECK: bb2:
207207
// CHECK: strong_retain %0 : $Cls
208-
// CHECK: cond_br undef, bb1, bb4
208+
// CHECK: cond_br undef, bb1, bb3
209209
// CHECK-LABEL: } // end sil function '$unmatched_rr_loop_early_exit'
210210
sil hidden @$unmatched_rr_loop_early_exit : $@convention(thin) (@owned Cls) -> () {
211211
bb0(%0 : $Cls):
@@ -233,10 +233,10 @@ bb3:
233233
// CHECK-LABEL: sil hidden @$matched_rr_loop_early_exit :
234234
// CHECK: bb1
235235
// CHECK: strong_retain %0 : $Cls
236-
// CHECK: cond_br undef, bb3, bb2
237-
// CHECK: bb3:
236+
// CHECK: cond_br undef, bb2, bb3
237+
// CHECK: bb2:
238238
// CHECK: strong_release %0 : $Cls
239-
// CHECK: cond_br undef, bb1, bb4
239+
// CHECK: cond_br undef, bb1, bb3
240240
// CHECK-LABEL: } // end sil function '$matched_rr_loop_early_exit'
241241
sil hidden @$matched_rr_loop_early_exit : $@convention(thin) (@owned Cls) -> () {
242242
bb0(%0 : $Cls):

0 commit comments

Comments
 (0)