[PATCH] D83765: [SVE] Add lowering for scalable vector fadd, fdiv, fmul and fsub operations.

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 14 07:39:14 PDT 2020


cameron.mcinally accepted this revision.
cameron.mcinally added a comment.
This revision is now accepted and ready to land.

LGTM

Digressing a bit, but I'd like to talk about unpacked integer vector operations at the next Sync-up call. It looks like they're currently promoted, e.g. (nxv2i32->nxv2i64). I'm wondering if that's the best option, or if handling them like the unpacked FP operations is better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83765





More information about the llvm-commits mailing list