Matheus Ferst <matheus.fe...@eldorado.org.br> writes: > The macro is missing a '{' after the if condition. Any use of REQUIRE_HV > would cause a compilation error. > > Fixes: fc34e81acd51 ("target/ppc: add macros to check privilege level") > Signed-off-by: Matheus Ferst <matheus.fe...@eldorado.org.br>
Reviewed-by: Fabiano Rosas <faro...@linux.ibm.com> > --- > target/ppc/translate.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/target/ppc/translate.c b/target/ppc/translate.c > index eaac8670b1..435066c4a3 100644 > --- a/target/ppc/translate.c > +++ b/target/ppc/translate.c > @@ -6545,12 +6545,12 @@ static int64_t dw_compose_ea(DisasContext *ctx, int x) > } \ > } while (0) > > -#define REQUIRE_HV(CTX) \ > - do { \ > - if (unlikely((CTX)->pr || !(CTX)->hv)) \ > - gen_priv_opc(CTX); \ > - return true; \ > - } \ > +#define REQUIRE_HV(CTX) \ > + do { \ > + if (unlikely((CTX)->pr || !(CTX)->hv)) { \ > + gen_priv_opc(CTX); \ > + return true; \ > + } \ > } while (0) > #else > #define REQUIRE_SV(CTX) do { gen_priv_opc(CTX); return true; } while (0)