Re: [PATCH v4 06/40] target/arm: Split out vae1_tlbmask, vmalle1_tlbmask

2019-12-03 Thread Richard Henderson
On 12/2/19 10:25 PM, Philippe Mathieu-Daudé wrote: > On 12/3/19 3:29 AM, Richard Henderson wrote: >> No functional change, but unify code sequences. >> >> Reviewed-by: Alex Bennée >> Signed-off-by: Richard Henderson > > Easier to review in 2 patches: vae1_tlbmask first, then vmalle1_tlbmask. Ok

Re: [PATCH v4 06/40] target/arm: Split out vae1_tlbmask, vmalle1_tlbmask

2019-12-02 Thread Philippe Mathieu-Daudé
On 12/3/19 3:29 AM, Richard Henderson wrote: No functional change, but unify code sequences. Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson Easier to review in 2 patches: vae1_tlbmask first, then vmalle1_tlbmask. If you need to respin, the 2 patches are welcome. Regardless: Revie

[PATCH v4 06/40] target/arm: Split out vae1_tlbmask, vmalle1_tlbmask

2019-12-02 Thread Richard Henderson
No functional change, but unify code sequences. Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson --- target/arm/helper.c | 118 ++-- 1 file changed, 37 insertions(+), 81 deletions(-) diff --git a/target/arm/helper.c b/target/arm/helper.c index 7