[PATCH] D104237: [RISCV][VP] Lower FP VP ISD nodes to RVV instructions

Wed Jun 16 02:45:07 PDT 2021

frasercrmck added a comment.

In D104237#2819994 <https://reviews.llvm.org/D104237#2819994>, @craig.topper wrote:

> Many targets scalarize frem on fixed vectors. It looks like frem on scalable vectors currently crashes on SVE because the type legalizer can't unroll it.

Indeed. I don't think we or any target support any level of scalable-vector `frem` right now, so I think this discussion is orthogonal to this VP patch. But I'll spend some time today seeing if there's anything we can do since it's come up.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104237