> LGTM. Committed, thanks Robin.
> Let's keep in mind that min/max will save us two insns(?) > and a conditional move would save us one. Got it, cmov is well designed for such case(s). Pan -----Original Message----- From: Robin Dapp <rdapp....@gmail.com> Sent: Friday, June 7, 2024 9:57 PM To: Li, Pan2 <pan2...@intel.com>; gcc-patches@gcc.gnu.org Cc: rdapp....@gmail.com; juzhe.zh...@rivai.ai; kito.ch...@gmail.com Subject: Re: [PATCH v3] RISC-V: Implement .SAT_SUB for unsigned scalar int LGTM. Let's keep in mind that min/max will save us two insns(?) and a conditional move would save us one. Regards Robin