about summary refs log tree commit diff
diff options
context:
space:
mode:
authorErik Desjardins <erikdesjardins@users.noreply.github.com>2020-06-15 00:50:56 -0400
committerErik Desjardins <erikdesjardins@users.noreply.github.com>2020-06-15 01:00:46 -0400
commit0906066ae7d8a5e217e8cbf7f17de78a00d4ed83 (patch)
treec137c28ac5fb8ed072e6c70a3c8a525ddfb21352
parent06e47688bf15d0215edbe05b21603062f6d2eb5d (diff)
downloadrust-0906066ae7d8a5e217e8cbf7f17de78a00d4ed83.tar.gz
rust-0906066ae7d8a5e217e8cbf7f17de78a00d4ed83.zip
Test that bounds checks are elided when slice len is checked up-front
-rw-r--r--src/test/codegen/issue-69101-bounds-check.rs26
1 files changed, 26 insertions, 0 deletions
diff --git a/src/test/codegen/issue-69101-bounds-check.rs b/src/test/codegen/issue-69101-bounds-check.rs
new file mode 100644
index 00000000000..cdbe51da03c
--- /dev/null
+++ b/src/test/codegen/issue-69101-bounds-check.rs
@@ -0,0 +1,26 @@
+// no-system-llvm
+// compile-flags: -O
+#![crate_type = "lib"]
+
+// CHECK-LABEL: @already_sliced_no_bounds_check
+#[no_mangle]
+pub fn already_sliced_no_bounds_check(a: &[u8], b: &[u8], c: &mut [u8]) {
+    // CHECK: slice_index_len_fail
+    // CHECK-NOT: panic_bounds_check
+    let _ = (&a[..2048], &b[..2048], &mut c[..2048]);
+    for i in 0..1024 {
+        c[i] = a[i] ^ b[i];
+    }
+}
+
+// make sure we're checking for the right thing: there can be a panic if the slice is too small
+// CHECK-LABEL: @already_sliced_bounds_check
+#[no_mangle]
+pub fn already_sliced_bounds_check(a: &[u8], b: &[u8], c: &mut [u8]) {
+    // CHECK: slice_index_len_fail
+    // CHECK: panic_bounds_check
+    let _ = (&a[..1023], &b[..2048], &mut c[..2048]);
+    for i in 0..1024 {
+        c[i] = a[i] ^ b[i];
+    }
+}