ByteAddressableBuffer: fix oob calculation being too strict

Firestar99 · Firestar99 · commit dc2b6de8e9ef · 2024-12-17T17:46:53.000+01:00
diff --git a/crates/spirv-std/src/byte_addressable_buffer.rs b/crates/spirv-std/src/byte_addressable_buffer.rs
@@ -71,7 +71,7 @@ fn bounds_check<T>(data: &[u32], byte_index: u32) {
         panic!("`byte_index` should be a multiple of 4");
     }
     let last_byte = byte_index + sizeof;
-    let len = data.len() as u32;
+    let len = data.len() as u32 * 4;
     if byte_index + sizeof > len {
         panic!(
             "index out of bounds: the len is {} but loading {} bytes at `byte_index` {} reads until {} (exclusive)",

Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ fn bounds_check<T>(data: &[u32], byte_index: u32) {`
`71`	`71`	panic!("`byte_index` should be a multiple of 4");
`72`	`72`	`}`
`73`	`73`	`let last_byte = byte_index + sizeof;`
`74`		`- let len = data.len() as u32;`
	`74`	`+ let len = data.len() as u32 * 4;`
`75`	`75`	`if byte_index + sizeof > len {`
`76`	`76`	`panic!(`
`77`	`77`	"index out of bounds: the len is {} but loading {} bytes at `byte_index` {} reads until {} (exclusive)",