mirrors/Odin
1
0
Fork 0
mirror of https://github.com/odin-lang/Odin.git synced 2026-02-14 23:33:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #4564 from gfaster/4553-regression-test

Browse Source 
Add regression test for #4553
This commit is contained in:
gingerBill
2024-12-06 14:20:51 +00:00
committed by GitHub
parent cf53404f5f 25ae3d0785
commit 62768ad857
1 changed files with 53 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

53
tests/internal/test_4553_matrix_align.odin Normal file
View File

@@ -0,0 +1,53 @@
package test_internal
import "core:testing"
// See: https://github.com/odin-lang/Odin/issues/4553
// It would be great if Odin had the ability to test against LLVM output since
// this test erronously passing is dependent on alignment of the stack.
//
// Right now, I am manually checking LLVM ir
@test
test_4553_matrix_align :: proc(t: ^testing.T) {
test_mat :: proc($T: typeid, $R: u32, $C: u32) {
when R * C <= 16 {
return_matrix :: proc() -> matrix[R, C]T {
ret : matrix[R, C]T
return ret
}
// the origin of the bug had to do with a temporary
// created by a function return being loaded with bad
// alignment. The bug only affected 4-element f32
// matrices, but it would be prudent to test more than
// that
_ = return_matrix() * [C]T{}
}
}
test_mat_set :: proc($T: typeid) {
test_mat_row :: proc($T: typeid, $R: u32) {
test_mat(T, R, 1)
test_mat(T, R, 2)
test_mat(T, R, 3)
test_mat(T, R, 4)
test_mat(T, R, 5)
test_mat(T, R, 6)
}
test_mat_row(T, 1)
test_mat_row(T, 2)
test_mat_row(T, 3)
test_mat_row(T, 4)
test_mat_row(T, 5)
test_mat_row(T, 6)
}
test_mat_set(f16)
test_mat_set(f32)
test_mat_set(f64)
test_mat_set(i8)
test_mat_set(i16)
test_mat_set(i32)
test_mat_set(i64)
test_mat_set(i128)
}