On 09/07/2025 09:28, Kyrylo Tkachov wrote:
Hi Alfie,

On 7 Jul 2025, at 10:46, Alfie Richards <alfie.richa...@arm.com> wrote:

Hello all,

This patch implements the couple of amin/amax instructions that are part of
SME2 + faminmax.

Regression testsed and bootstrapped for Aarch64.

Thanks,
Alfie

-- >8 --

Implements the sme2+faminmax svamin and svamax intrinsics.

gcc/ChangeLog:

* config/aarch64/aarch64-sme.md (@aarch64_sme_<faminmax_uns_op><mode>):
New patterns.
* config/aarch64/aarch64-sve-builtins-sme.def (svamin): New intrinsics.
(svamax): New intrinsics.
* config/aarch64/aarch64-sve-builtins-sve2.cc (class faminmaximpl): New
class.
(svamin): New function.
(svamax): New function.


+#define REQUIRED_EXTENSIONS streaming_only (AARCH64_FL_SME2)

Do these also need an FAMINMAX guard?
Looks ok to me otherwise.
Thanks,
Kyrill

Hi Kyrill,

Ah yes they do, thank you!
Fixed and committed.

Thanks,
Alfie

+DEF_SME_FUNCTION_GS (svamin, binary_opt_single_n, all_float, x24, none)
+DEF_SME_FUNCTION_GS (svamax, binary_opt_single_n, all_float, x24, none)
+#undef REQUIRED_EXTENSIONS

Reply via email to