On 7/30/25 11:22 PM, pan2...@intel.com wrote:
From: Pan Li <pan2...@intel.com>

The previous code-gen of scalar unsigned SAT_MUL, aka usmul.
Leverage the mulhs by mistake, it should be mulhu for the
hight bit result of mul.  Thus, this patch would like to make
it correct.

gcc/ChangeLog:

        * config/riscv/riscv.cc (riscv_expand_xmode_usmul): Take
        umulhu for high bits mul result.

gcc/testsuite/ChangeLog:

        * gcc.target/riscv/sat/sat_u_mul-1-u32-from-u64.c: Add mulhu
        asm check.
        * gcc.target/riscv/sat/sat_u_mul-1-u64-from-u128.c: Ditto.
OK.  Any change we could get run test for this in addition to the scanners?

jeff

Reply via email to