Results 144 comments of gingerBill

So this is actually tricky to handle correctly because it has to do a best fit search, and currently it does the "first fit" search. In the case other structs,...

With regards to allowing `#align` on a `#packed` struct, what should be the size of this data structure? ```odin blah :: struct #packed #align 2 {x: [3]u8} size_of(blah) == ???...

@JesseRMeyer Because there is a rule in most programming languages that `size_of(T) % align_of(T) == 0 `. Making it `3` would break this rule.

From what you re saying, it ought to be an error.

This problem has now been solved with `#field_align(N)`. `#field_align(N)` will set the minimum alignment for each field to `N` or its natural alignment, whichever is larger. ```odin Foo :: struct...

Did you even test this code before you wrote it?

Which LLVM 11 version are you using? 11.1.0 or 11.0.x? We highly recommend using LLVM 11.1.0 not 11.0.x.

I cannot replicate this bug any more.

So the first error is a bug and shouldn't show. The second isn't a bug because you've overridden the current scope's `Foo` with that field and it doesn't know which...

Couldn't we overload the collection system to do this? Like how `system` is reserved. We could have `system_app` and `system_bin`.