diff options
| author | BALATON Zoltan <balaton@eik.bme.hu> | 2024-05-27 01:12:40 +0200 |
|---|---|---|
| committer | Nicholas Piggin <npiggin@gmail.com> | 2024-07-26 09:51:33 +1000 |
| commit | 3208c36ad34213eee36b1427d8cb944cfa0a192c (patch) | |
| tree | 67460b38c0c0f7a0116d2070709345c88a978264 /target/ppc/mmu_common.c | |
| parent | f6b50257c7c7297be6bcafe8ff977a38c965d0c0 (diff) | |
| download | qemu-3208c36ad34213eee36b1427d8cb944cfa0a192c.zip qemu-3208c36ad34213eee36b1427d8cb944cfa0a192c.tar.gz qemu-3208c36ad34213eee36b1427d8cb944cfa0a192c.tar.bz2 | |
target/ppc/mmu_common.c: Remove another single use local variable
In ppc6xx_tlb_pte_check() the pteh variable is used only once to
compare to the h parameter of the function. Inline its value and use
pteh name for the function parameter which is more descriptive.
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Diffstat (limited to 'target/ppc/mmu_common.c')
| -rw-r--r-- | target/ppc/mmu_common.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/target/ppc/mmu_common.c b/target/ppc/mmu_common.c index 16578f7..b21f522 100644 --- a/target/ppc/mmu_common.c +++ b/target/ppc/mmu_common.c @@ -95,15 +95,14 @@ int ppc6xx_tlb_getnum(CPUPPCState *env, target_ulong eaddr, } static int ppc6xx_tlb_pte_check(mmu_ctx_t *ctx, target_ulong pte0, - target_ulong pte1, int h, + target_ulong pte1, int pteh, MMUAccessType access_type) { - int ret, pteh, pp; + int ret, pp; ret = -1; /* Check validity and table match */ - pteh = (pte0 >> 6) & 1; - if (pte_is_valid(pte0) && h == pteh) { + if (pte_is_valid(pte0) && ((pte0 >> 6) & 1) == pteh) { /* Check vsid & api */ pp = pte1 & 0x00000003; if ((pte0 & PTE_PTEM_MASK) == ctx->ptem) { |