Re: [PATCH v3 08/60] target/arm: Change DisasContext.thumb to bool

[Richard Henderson]

On 4/22/22 07:04, Peter Maydell wrote:
> On Fri, 22 Apr 2022 at 15:01, Alex Bennée <alex.bennee@linaro.org> wrote:
> > Richard Henderson <richard.henderson@linaro.org> writes:
> >
> > > Bool is a more appropriate type for this value.
> > > Move the member down in the struct to keep the
> > > bool type members together and remove a hole.
> >
> > Does gcc even attempt to pack bools? Aren't they basically int types?
>
> It's impdef, I think, but it'll typically be a 1 byte integer
> rather than a 4 byte integer, with the usual struct packing
> rules for 1 byte type sizes.

Yep, it's 1 byte for all extant abis except macos where it's 4.


r~