[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 6/7] target/arm: Implement v8.3 FPAC and FPACCOMBINE
From: |
Aaron Lindsay |
Subject: |
Re: [PATCH v2 6/7] target/arm: Implement v8.3 FPAC and FPACCOMBINE |
Date: |
Wed, 22 Mar 2023 16:33:04 -0400 |
On Feb 22 11:37, Richard Henderson wrote:
> On 2/22/23 09:35, Aaron Lindsay wrote:
> > @@ -406,6 +421,16 @@ static uint64_t pauth_auth(CPUARMState *env, uint64_t
> > ptr, uint64_t modifier,
> > uint64_t xor_mask = MAKE_64BIT_MASK(bot_bit, top_bit - bot_bit +
> > 1) &
> > ~MAKE_64BIT_MASK(55, 1);
> > result = ((ptr ^ pac) & xor_mask) | (ptr & ~xor_mask);
> > + if (cpu_isar_feature(aa64_fpac_combine, env_archcpu(env)) ||
> > + (cpu_isar_feature(aa64_fpac, env_archcpu(env)) &&
> > + !is_combined)) {
>
> Indentation is off.
I pulled `env_archcpu(env)` out of this if-statement in my latest
patchset in addition to the indentation, but am not confident I have
done what you intended. The QEMU Coding Style guide doesn't seem to
address longer statements like this in its section on indentation, so I
attempted to follow other examples in the code, but I'll take further
direction here.
-Aaron
- Re: [PATCH v2 6/7] target/arm: Implement v8.3 FPAC and FPACCOMBINE,
Aaron Lindsay <=