[PATCH] D89162: [SVE] Lower fixed length VECREDUCE_SEQ_FADD operation

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 14:25:31 PDT 2020


cameron.mcinally added a comment.

> [1] I just wanted to highlight my previous VBITS_EQ_256-COUNT-33: fadd comment as this gives us a bit more test coverage and is something that will obviously fail (in a good way) when the splitting work is available.

Got it. Also added a FIXME to remind us to move the `useSVEForFixedLengthVectors*()` functions back to private scope when the reduction legalization is complete.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89162



More information about the llvm-commits mailing list