On 6/14/21 5:58 PM, Richard Henderson wrote: > On 6/14/21 2:41 AM, Philippe Mathieu-Daudé wrote: >> On 6/14/21 10:37 AM, Richard Henderson wrote: >>> Implement the new semantics in the fallback expansion. >>> Change all callers to supply the flags that keep the >>> semantics unchanged locally. >>> >>> Signed-off-by: Richard Henderson <richard.hender...@linaro.org> >>> --- >>> include/tcg/tcg-op.h | 8 +-- >>> target/arm/translate-a64.c | 12 ++-- >>> target/arm/translate.c | 2 +- >>> target/i386/tcg/translate.c | 2 +- >>> target/mips/tcg/mxu_translate.c | 2 +- >>> target/s390x/translate.c | 4 +- >>> target/sh4/translate.c | 2 +- >> >> Various REV 16/32, would it be useful to have it as a TCG opcode? > > Which operation are you proposing as tcg opcode? The per-halfword swap > akin to mips wsbh? Yes, that operation also appears in arm (rev16) and > ppc (brh). So it's a reasonable thing to do.
and REV32 for PPC BRW? Another I noticed is popcnt.