[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qemu/int128: Re-shuffle Int128Alias members
|
From: |
Alex Bennée |
|
Subject: |
Re: [PATCH] qemu/int128: Re-shuffle Int128Alias members |
|
Date: |
Tue, 02 May 2023 09:44:11 +0100 |
|
User-agent: |
mu4e 1.11.4; emacs 29.0.90 |
Richard Henderson <richard.henderson@linaro.org> writes:
> Clang 14, with --enable-tcg-interpreter errors with
>
> include/qemu/int128.h:487:16: error: alignment of field 'i' (128 bits)
> does not match the alignment of the first field in transparent union;
> transparent_union attribute ignored [-Werror,-Wignored-attributes]
> __int128_t i;
> ^
> include/qemu/int128.h:486:12: note: alignment of first field is 64 bits
> Int128 s;
> ^
> 1 error generated.
>
> By placing the __uint128_t member first, this is avoided.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
--
Alex Bennée
Virtualisation Tech Lead @ Linaro