[PATCH] D87843: [SVE] Change definition of reduction ISD nodes to have an SVE vector result type.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 10:25:38 PDT 2020


paulwalker-arm created this revision.
Herald added subscribers: llvm-commits, psnobl, hiraditya, kristof.beyls, tschuett.
Herald added a reviewer: rengolin.
Herald added a reviewer: efriedma.
Herald added a project: LLVM.
paulwalker-arm requested review of this revision.

The current nodes, AArch64::SMAXV_PRED for example, are defined to
return a NEON vector result.  This is incorrect because they modify
the complete SVE register and are thus changed to represent such.

This patch also adds nodes for UADDV_PRED and SADDV_PRED, which
unifies the handling of all SVE reductions.

NOTE: Floating-point reductions are already implemented correctly,
so this patch is essentially making everything consistent with those.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D87843

Files:
  llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
  llvm/lib/Target/AArch64/AArch64ISelLowering.h
  llvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
  llvm/lib/Target/AArch64/SVEInstrFormats.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87843.292556.patch
Type: text/x-patch
Size: 14561 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200917/caa24e82/attachment.bin>


More information about the llvm-commits mailing list