On 10/01/2016 05:05 AM, Artyom Tarasenko wrote:
+uint64_t mask = 1ULL + ~(8192ULL << 3 * TTE_PGSIZE(tlb->tte));
Not the first time it has appeared in this patch series, but I find the "1 + ~"
idiom to be strange. Surely plain old "-" is clearer.
r~
Signed-off-by: Artyom Tarasenko
---
target-sparc/mmu_helper.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/target-sparc/mmu_helper.c b/target-sparc/mmu_helper.c
index bef63f8..33b48eb 100644
--- a/target-sparc/mmu_helper.c
+++ b/target-sparc/mmu_helper.c