[PATCH] D152370: [Intrinsic] Introduce reduction intrinsics for minimum/maximum

Mon Jun 12 09:06:23 PDT 2023

anna added a comment.

In D152370#4411543 <https://reviews.llvm.org/D152370#4411543>, @dmgreen wrote:

>> I haven't added any custom actions for any targets, since that is out of the scope of this patch (any improvements can be done over this code).
>
> Sounds good. I'm happy to look into the legal instructions for AArch64. As far as I understand fminimum/fmaximum were relatively uncommon so there may be holes in the lowering. Let me know if I can help.

Thanks @dmgreen. As far as I can tell, the patch I've linked to this one should close the legalization bugs showing up in AArch64 for fmaximum/fminimum.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152370/new/

https://reviews.llvm.org/D152370