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

Add extra max alignment parameter for metrics (specifically for SIMD)

Browse Source
This commit is contained in:
gingerBill
2022-08-24 12:58:16 +01:00
parent f1ffd90294
commit 4ba486baa2
2 changed files with 31 additions and 67 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/types.cpp
View File

@@ -3556,7 +3556,7 @@ i64 type_align_of_internal(Type *t, TypePath *path) {
case Type_SimdVector: {
// IMPORTANT TODO(bill): Figure out the alignment of vector types
return gb_clamp(next_pow2(type_size_of_internal(t, path)), 1, build_context.max_align*2);
return gb_clamp(next_pow2(type_size_of_internal(t, path)), 1, build_context.max_simd_align*2);
}
case Type_Matrix:
@@ -3571,22 +3571,9 @@ i64 type_align_of_internal(Type *t, TypePath *path) {
return build_context.word_size;
}
i64 max_alignment = build_context.word_size;
if (is_arch_wasm()) {
// NOTE(bill): wasm32 with LLVM defines its default datalayout as:
//
// e-m:e-p:32:32-i64:64-n32:64-S128
//
// This means that the alignment of a 64-bit type is 64-bits and not 32-bits like
// on other 32-bit architectures
//
// See: https://github.com/WebAssembly/tool-conventions/blob/main/BasicCABI.md
max_alignment = 8;
}
// NOTE(bill): Things that are bigger than build_context.word_size, are actually comprised of smaller types
// TODO(bill): Is this correct for 128-bit types (integers)?
return gb_clamp(next_pow2(type_size_of_internal(t, path)), 1, max_alignment);
return gb_clamp(next_pow2(type_size_of_internal(t, path)), 1, build_context.max_align);
}
i64 *type_set_offsets_of(Slice<Entity *> const &fields, bool is_packed, bool is_raw_union) {