Spencer Abson <spencer.ab...@arm.com> writes:
> diff --git a/gcc/config/aarch64/iterators.md b/gcc/config/aarch64/iterators.md
> index 146453b0516..b4ebc493a4d 100644
> --- a/gcc/config/aarch64/iterators.md
> +++ b/gcc/config/aarch64/iterators.md
> @@ -479,13 +479,25 @@
>  ;; All fully-packed SVE integer and Advanced SIMD integer modes.
>  (define_mode_iterator SVE_ASIMD_FULL_I [SVE_FULL_I VDQ_I])
>  
> -;; All fully-packed SVE floating-point vector modes.
> +;; All fully-packed SVE (non-BF16) floating-point vector modes.
>  (define_mode_iterator SVE_FULL_F [VNx8HF VNx4SF VNx2DF])
>  
> +;; All partial SVE (non-BF16) floating-point vector modes.
> +(define_mode_iterator SVE_PARTIAL_F [VNx2HF VNx4HF VNx2SF])
> +
> +;; All SVE (non-BF16) floating-point vector modes.
> +(define_mode_iterator SVE_F [SVE_PARTIAL_F SVE_FULL_F])
> +

Obviously personal preference, sorry, but: now that these are not
"all" modes, I think we should drop the "all":

;; Fully-packed SVE floating-point vector modes, excluding BF16.

and similarly for the others.  OK with that change, thanks.

Richard

Reply via email to