[PATCH] D89070: [AArch64][GlobalISel] Add basic legalizer rules for supported add/fadd reductions.

Thu Oct 8 20:07:57 PDT 2020

aemerson updated this revision to Diff 297110.
aemerson added a comment.

Remove v2s16 FADD case because we can't actually support that with a single instruction. Exactly how we're going to decide what's "legal" and what will need to be legalized first isn't totally clear yet. Using a criteria of a single selected instruction seems a good start.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89070

Files:
  llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-reduce-add.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-reduce-fadd.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D89070.297110.patch
Type: text/x-patch
Size: 7763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201009/188ed735/attachment-0001.bin>