| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 1 | ; RUN: opt < %s -rewrite-statepoints-for-gc -S 2>&1 | FileCheck %s |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 2 | |
| 3 | ; The rewriting needs to make %obj loop variant by inserting a phi |
| 4 | ; of the original value and it's relocation. |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 5 | |
| 6 | declare i64 addrspace(1)* @generate_obj() "gc-leaf-function" |
| 7 | |
| 8 | declare void @use_obj(i64 addrspace(1)*) "gc-leaf-function" |
| 9 | |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 10 | define void @def_use_safepoint() gc "statepoint-example" { |
| 11 | ; CHECK-LABEL: def_use_safepoint |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 12 | ; CHECK: phi i64 addrspace(1)* |
| 13 | ; CHECK-DAG: [ %obj.relocated.casted, %loop ] |
| 14 | ; CHECK-DAG: [ %obj, %entry ] |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 15 | entry: |
| 16 | %obj = call i64 addrspace(1)* @generate_obj() |
| 17 | br label %loop |
| 18 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 19 | loop: ; preds = %loop, %entry |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 20 | call void @use_obj(i64 addrspace(1)* %obj) |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 21 | call void @do_safepoint() [ "deopt"(i32 0, i32 -1, i32 0, i32 0, i32 0) ] |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 22 | br label %loop |
| 23 | } |
| 24 | |
| 25 | declare void @do_safepoint() |
| 26 | |
| 27 | declare void @parse_point(i64 addrspace(1)*) |
| 28 | |
| 29 | define i64 addrspace(1)* @test1(i32 %caller, i8 addrspace(1)* %a, i8 addrspace(1)* %b, i32 %unknown) gc "statepoint-example" { |
| 30 | ; CHECK-LABEL: test1 |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 31 | entry: |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 32 | br i1 undef, label %left, label %right |
| 33 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 34 | left: ; preds = %entry |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 35 | ; CHECK: left: |
| 36 | ; CHECK-NEXT: %a.cast = bitcast i8 addrspace(1)* %a to i64 addrspace(1)* |
| 37 | ; CHECK-NEXT: [[CAST_L:%.*]] = bitcast i8 addrspace(1)* %a to i64 addrspace(1)* |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 38 | ; Our safepoint placement pass calls removeUnreachableBlocks, which does a bunch |
| 39 | ; of simplifications to branch instructions. This bug is visible only when |
| 40 | ; there are multiple branches into the same block from the same predecessor, and |
| 41 | ; the following ceremony is to make that artefact survive a call to |
| 42 | ; removeUnreachableBlocks. As an example, "br i1 undef, label %merge, label %merge" |
| 43 | ; will get simplified to "br label %merge" by removeUnreachableBlocks. |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 44 | %a.cast = bitcast i8 addrspace(1)* %a to i64 addrspace(1)* |
| 45 | switch i32 %unknown, label %right [ |
| 46 | i32 0, label %merge |
| 47 | i32 1, label %merge |
| 48 | i32 5, label %merge |
| 49 | i32 3, label %right |
| 50 | ] |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 51 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 52 | right: ; preds = %left, %left, %entry |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 53 | ; CHECK: right: |
| 54 | ; CHECK-NEXT: %b.cast = bitcast i8 addrspace(1)* %b to i64 addrspace(1)* |
| 55 | ; CHECK-NEXT: [[CAST_R:%.*]] = bitcast i8 addrspace(1)* %b to i64 addrspace(1)* |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 56 | %b.cast = bitcast i8 addrspace(1)* %b to i64 addrspace(1)* |
| 57 | br label %merge |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 58 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 59 | merge: ; preds = %right, %left, %left, %left |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 60 | ; CHECK: merge: |
| Philip Reames | fa2c630 | 2015-07-24 19:01:39 +0000 | [diff] [blame] | 61 | ; CHECK-NEXT: %value.base = phi i64 addrspace(1)* [ [[CAST_L]], %left ], [ [[CAST_L]], %left ], [ [[CAST_L]], %left ], [ [[CAST_R]], %right ], !is_base_value !0 |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 62 | %value = phi i64 addrspace(1)* [ %a.cast, %left ], [ %a.cast, %left ], [ %a.cast, %left ], [ %b.cast, %right ] |
| 63 | call void @parse_point(i64 addrspace(1)* %value) [ "deopt"(i32 0, i32 0, i32 0, i32 0, i32 0) ] |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 64 | ret i64 addrspace(1)* %value |
| 65 | } |
| 66 | |
| 67 | ;; The purpose of this test is to ensure that when two live values share a |
| 68 | ;; base defining value with inherent conflicts, we end up with a *single* |
| 69 | ;; base phi/select per such node. This is testing an optimization, not a |
| 70 | ;; fundemental correctness criteria |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 71 | define void @test2(i1 %cnd, i64 addrspace(1)* %base_obj, i64 addrspace(1)* %base_arg2) gc "statepoint-example" { |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 72 | ; CHECK-LABEL: @test2 |
| 73 | entry: |
| 74 | %obj = getelementptr i64, i64 addrspace(1)* %base_obj, i32 1 |
| 75 | br label %loop |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 76 | ; CHECK-LABEL: loop |
| Philip Reames | fa2c630 | 2015-07-24 19:01:39 +0000 | [diff] [blame] | 77 | ; CHECK: %current.base = phi i64 addrspace(1)* |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 78 | ; CHECK-DAG: [ %base_obj, %entry ] |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 79 | |
| Philip Reames | df1ef08 | 2015-04-10 22:53:14 +0000 | [diff] [blame] | 80 | ; Given the two selects are equivelent, so are their base phis - ideally, |
| 81 | ; we'd have commoned these, but that's a missed optimization, not correctness. |
| Philip Reames | fa2c630 | 2015-07-24 19:01:39 +0000 | [diff] [blame] | 82 | ; CHECK-DAG: [ [[DISCARD:%.*.base.relocated.casted]], %loop ] |
| 83 | ; CHECK-NOT: extra.base |
| Philip Reames | 79fa9b7 | 2016-02-22 20:45:56 +0000 | [diff] [blame] | 84 | ; CHECK: next.base = select |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 85 | ; CHECK: next = select |
| Philip Reames | fa2c630 | 2015-07-24 19:01:39 +0000 | [diff] [blame] | 86 | ; CHECK: extra2.base = select |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 87 | ; CHECK: extra2 = select |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 88 | ; CHECK: statepoint |
| 89 | ;; Both 'next' and 'extra2' are live across the backedge safepoint... |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 90 | |
| 91 | loop: ; preds = %loop, %entry |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 92 | %current = phi i64 addrspace(1)* [ %obj, %entry ], [ %next, %loop ] |
| 93 | %extra = phi i64 addrspace(1)* [ %obj, %entry ], [ %extra2, %loop ] |
| 94 | %nexta = getelementptr i64, i64 addrspace(1)* %current, i32 1 |
| 95 | %next = select i1 %cnd, i64 addrspace(1)* %nexta, i64 addrspace(1)* %base_arg2 |
| 96 | %extra2 = select i1 %cnd, i64 addrspace(1)* %nexta, i64 addrspace(1)* %base_arg2 |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 97 | call void @foo() [ "deopt"(i32 0, i32 -1, i32 0, i32 0, i32 0) ] |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 98 | br label %loop |
| 99 | } |
| 100 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 101 | define i64 addrspace(1)* @test3(i1 %cnd, i64 addrspace(1)* %obj, i64 addrspace(1)* %obj2) gc "statepoint-example" { |
| Philip Reames | abcdc5e | 2015-08-27 01:02:28 +0000 | [diff] [blame] | 102 | ; CHECK-LABEL: @test3 |
| 103 | entry: |
| 104 | br i1 %cnd, label %merge, label %taken |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 105 | |
| 106 | taken: ; preds = %entry |
| Philip Reames | abcdc5e | 2015-08-27 01:02:28 +0000 | [diff] [blame] | 107 | br label %merge |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 108 | |
| 109 | merge: ; preds = %taken, %entry |
| Philip Reames | abcdc5e | 2015-08-27 01:02:28 +0000 | [diff] [blame] | 110 | ; CHECK-LABEL: merge: |
| Philip Reames | 79fa9b7 | 2016-02-22 20:45:56 +0000 | [diff] [blame] | 111 | ; CHECK-NEXT: phi |
| 112 | ; CHECK-NEXT: phi |
| Philip Reames | abcdc5e | 2015-08-27 01:02:28 +0000 | [diff] [blame] | 113 | ; CHECK-NEXT: gc.statepoint |
| 114 | %bdv = phi i64 addrspace(1)* [ %obj, %entry ], [ %obj2, %taken ] |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 115 | call void @foo() [ "deopt"(i32 0, i32 -1, i32 0, i32 0, i32 0) ] |
| Philip Reames | abcdc5e | 2015-08-27 01:02:28 +0000 | [diff] [blame] | 116 | ret i64 addrspace(1)* %bdv |
| 117 | } |
| 118 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 119 | define i64 addrspace(1)* @test4(i1 %cnd, i64 addrspace(1)* %obj, i64 addrspace(1)* %obj2) gc "statepoint-example" { |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 120 | ; CHECK-LABEL: @test4 |
| 121 | entry: |
| 122 | br i1 %cnd, label %merge, label %taken |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 123 | |
| 124 | taken: ; preds = %entry |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 125 | br label %merge |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 126 | |
| 127 | merge: ; preds = %taken, %entry |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 128 | ; CHECK-LABEL: merge: |
| Philip Reames | 79fa9b7 | 2016-02-22 20:45:56 +0000 | [diff] [blame] | 129 | ; CHECK-NEXT: phi |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 130 | ; CHECK-NEXT: gc.statepoint |
| 131 | %bdv = phi i64 addrspace(1)* [ %obj, %entry ], [ %obj, %taken ] |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 132 | call void @foo() [ "deopt"(i32 0, i32 -1, i32 0, i32 0, i32 0) ] |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 133 | ret i64 addrspace(1)* %bdv |
| 134 | } |
| 135 | |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 136 | define i64 addrspace(1)* @test5(i1 %cnd, i64 addrspace(1)* %obj, i64 addrspace(1)* %obj2) gc "statepoint-example" { |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 137 | ; CHECK-LABEL: @test5 |
| 138 | entry: |
| 139 | br label %merge |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 140 | |
| 141 | merge: ; preds = %merge, %entry |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 142 | ; CHECK-LABEL: merge: |
| Philip Reames | 79fa9b7 | 2016-02-22 20:45:56 +0000 | [diff] [blame] | 143 | ; CHECK-NEXT: phi |
| 144 | ; CHECK-NEXT: phi |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 145 | ; CHECK-NEXT: br i1 |
| 146 | %bdv = phi i64 addrspace(1)* [ %obj, %entry ], [ %obj2, %merge ] |
| 147 | br i1 %cnd, label %merge, label %next |
| Sanjoy Das | 0407108 | 2016-01-29 00:28:57 +0000 | [diff] [blame] | 148 | |
| 149 | next: ; preds = %merge |
| 150 | call void @foo() [ "deopt"(i32 0, i32 -1, i32 0, i32 0, i32 0) ] |
| Philip Reames | dab35f3 | 2015-09-02 21:11:44 +0000 | [diff] [blame] | 151 | ret i64 addrspace(1)* %bdv |
| 152 | } |
| 153 | |
| Philip Reames | a5aeaf4 | 2015-02-28 00:20:48 +0000 | [diff] [blame] | 154 | declare void @foo() |