diff options
| author | BALATON Zoltan <balaton@eik.bme.hu> | 2024-05-13 01:28:06 +0200 |
|---|---|---|
| committer | Nicholas Piggin <npiggin@gmail.com> | 2024-05-24 09:43:12 +1000 |
| commit | 950251ee7b459d160fd3adfa2792326aa29b5d6c (patch) | |
| tree | 700bd4d85d7db2dcb947044eae3ac3db60d79d56 | |
| parent | e89b0629b99e3df96ef1c190139f77169c58d27b (diff) | |
| download | focaccia-qemu-950251ee7b459d160fd3adfa2792326aa29b5d6c.tar.gz focaccia-qemu-950251ee7b459d160fd3adfa2792326aa29b5d6c.zip | |
target/ppc/mmu-radix64.c: Drop a local variable
The value is only used once so no need to introduce a local variable for it. Reviewed-by: Nicholas Piggin <npiggin@gmail.com> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
| -rw-r--r-- | target/ppc/mmu-radix64.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c index fefa55a5f1..c1e4f00335 100644 --- a/target/ppc/mmu-radix64.c +++ b/target/ppc/mmu-radix64.c @@ -185,7 +185,6 @@ static bool ppc_radix64_check_prot(PowerPCCPU *cpu, MMUAccessType access_type, int mmu_idx, bool partition_scoped) { CPUPPCState *env = &cpu->env; - int need_prot; /* Check Page Attributes (pte58:59) */ if ((pte & R_PTE_ATT) == R_PTE_ATT_NI_IO && access_type == MMU_INST_FETCH) { @@ -210,8 +209,8 @@ static bool ppc_radix64_check_prot(PowerPCCPU *cpu, MMUAccessType access_type, } /* Check if requested access type is allowed */ - need_prot = prot_for_access_type(access_type); - if (need_prot & ~*prot) { /* Page Protected for that Access */ + if (prot_for_access_type(access_type) & ~*prot) { + /* Page Protected for that Access */ *fault_cause |= access_type == MMU_INST_FETCH ? SRR1_NOEXEC_GUARD : DSISR_PROTFAULT; return true; |