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]>

Reply via email to