Veikka Tuominen

Results 363 comments of Veikka Tuominen

Caused by `#define __alignof(__t)`, does not happen with https://github.com/ziglang/translate-c/

Yes if this diff doesn't fix it: ```diff diff --git a/lib/compiler/aro_translate_c.zig b/lib/compiler/aro_translate_c.zig index 9485bc79d2..e064cfa345 100644 --- a/lib/compiler/aro_translate_c.zig +++ b/lib/compiler/aro_translate_c.zig @@ -1206,7 +1206,7 @@ pub const PatternList = struct { ///...

I don't remember any reason not to store the value and would have expected it to already be done. At least changing it is easy.

The var arg instructions in LLVM don't work properly on most targets because clang doesn't use them. Reimplementing them ourselves like clang does would fix this as well.

That's what I meant by clang not using the `va_arg` instruction, I ran into it when originally implementing the feature and it's the reason why `VaList` is disabled on many...

Like tau-dev said the issue is the type being `comptime_int`: https://github.com/ziglang/zig/blob/f2bf6c1b11702179329a4693cea429d550f519e1/src/Sema.zig#L6716 Using `const var_one: i32 = 0x123456;` to match calling `varOne` has debug info as expected. Using `comptime_int` could be...

Only the third example in #19497 crashes now but it is different from this. The other two are the same as this one.

There are probably other cases where the type is printed but just matching Clang is also fine.

This still doesn't cover every case: ```c void foo(void) { const int arr[2][2] = { 0, 1, 2, 3}; arr[0][0] = 1; } ``` This is quite close to being...

> Tbh I want to finish it myself. That's fine too. > To save time, it would be great if you listed all the code snippets that should issue the...