[PATCH] D95870: [RISCV] Support the scalable-vector fadd reduction intrinsic

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 3 03:26:35 PST 2021


frasercrmck added a comment.

In D95870#2536916 <https://reviews.llvm.org/D95870#2536916>, @jrtc27 wrote:

> If something's not yet supported due to a spec deficiency that should be documented in the code with a link to the outstanding spec issue.

That's a good idea. My first thought would be `RISCVTTIImpl::shouldExpandReduction`, not that it can actually expand it right now? Regardless of our intentions, if we find such a reduction in the IR we crash.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95870



More information about the llvm-commits mailing list