[PATCH] D159190: [RISCV]Combine to make the fixed vector vadd of splats are scalarized

Wed Aug 30 06:57:52 PDT 2023

liaolucy added a comment.

In D159190#4628179 <https://reviews.llvm.org/D159190#4628179>, @luke wrote:

> Thanks for the patch. Did you take a look at scalarizeBinOpOfSplats in DAGCombiner? I think that might be more generic, it's the combine that kicks in for the scalable vector test case in that issue. I was wondering if it would be possible to get it to work on fixed vectors too.

I have a question, this ir has been optimized in the opt phase. https://godbolt.org/z/MPvvTG5dT  Maybe we don't need another combine in codegen?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159190