Skip to content

Commit

Permalink
tests: accept llvm intrinsic in align-checking test
Browse files Browse the repository at this point in the history
This changed in upstream change https://reviews.llvm.org/D98152 (aka
llvm/llvm-project@a266af7)
wherein LLVM got smarter about using intrinsics. As best I can tell the
change I've made here preserves the intent of the test on LLVM 14 and
before while also passing on LLVM 15 and later.
  • Loading branch information
durin42 committed Mar 1, 2022
1 parent f0c4da4 commit 26c5d21
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/test/codegen/dst-vtable-align-nonzero.rs
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ pub fn eliminates_runtime_check_when_align_1(
x: &Struct<WrapperWithAlign1<dyn Trait>>
) -> &WrapperWithAlign1<dyn Trait> {
// CHECK: load [[USIZE:i[0-9]+]], {{.+}} !range [[RANGE_META:![0-9]+]]
// CHECK-NOT: tail call i64 @llvm.umax.i64
// CHECK-NOT: icmp
// CHECK-NOT: select
// CHECK: ret
Expand All @@ -36,8 +37,7 @@ pub fn does_not_eliminate_runtime_check_when_align_2(
x: &Struct<WrapperWithAlign2<dyn Trait>>
) -> &WrapperWithAlign2<dyn Trait> {
// CHECK: [[X0:%[0-9]+]] = load [[USIZE]], {{.+}} !range [[RANGE_META]]
// CHECK: [[X1:%[0-9]+]] = icmp {{.+}} [[X0]]
// CHECK: [[X2:%[0-9]+]] = select {{.+}} [[X1]]
// CHECK: {{icmp|llvm.umax.i64}}
// CHECK: ret
&x.dst
}
Expand Down

0 comments on commit 26c5d21

Please sign in to comment.