On Mon, May 09, 2022 at 01:24:08PM +0100, Andrew Cooper wrote: > 'int' bitfields in particular have implementation defined behaviour under gcc > and can change signed-ness with -funsigned-bitfields. > > There is no need for low_bit_was_clear to be a bitfield in the first place; it > is only used as a boolean. Doing so even improves the code generation in > sh_emulate_map_dest() to avoid emitting a merge with structure padding. > > Spotted by Eclair MISRA scanner. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Reviewed-by: Roger Pau Monné <roger....@citrix.com> Thanks.