On 2/7/20 5:40 PM, Peter Maydell wrote: > On Mon, 3 Feb 2020 at 14:47, Richard Henderson > <richard.hender...@linaro.org> wrote: >> >> The CPSR_USER mask for IS_USER already avoids all of the RES0 >> bits as per aarch32_cpsr_valid_mask. Fix up the formatting. > > CPSR_USER includes CPSR_Q and CPSR_GE, which might be RES0 > depending on feature bit settings.
Oops, yes. > Diff made a bit of a mess of this patch -- I think it would > be easier to understand if the reformatting to add {} was > separate from the code change. Because of the above, I'll probably drop all of this except for the formatting fix. r~