On Wed, 2025-10-22 at 17:07 +0800, Jiahao Xu wrote:
> +machine_mode
> +loongarch_split_reduction (machine_mode mode)
> +{
> + switch (mode)
> + {
> + case E_V4DImode:
> + return V2DImode;
> + case E_V8SImode:
> + return V4SImode;
> + case E_V16HImode:
> + return V8HImode;
> + case E_V32QImode:
> + return V16QImode;
> + case E_V8SFmode:
> + return V4SFmode;
> + case E_V4DFmode:
> + return V2DFmode;
> + default:
> + return mode;
> + }
> +}
IMO this would be better:
if (LSX_SUPPORTED_MODE_P (mode))
return mode;
return mode_for_vector (as_a <scalar_mode> (GET_MODE_INNER (mode)),
GET_MODE_NUNITS (mode) / 2).require ();
--
Xi Ruoyao <[email protected]>