[PATCH] D159190: [RISCV]Combine to make the fixed vector vadd of splats are scalarized
Luke Lau via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 1 08:09:31 PDT 2023
luke added a comment.
In D159190#4634302 <https://reviews.llvm.org/D159190#4634302>, @luke wrote:
> But on RISC-V we'll need a vmv.x.s, so I don't think that qualifies as "free". But this is only if it's coming from a vector_shuffle if I understand correctly: build_vector splats and splat_vector splats should indeed be free because the extract_vector_elt will be combined away.
I gave this a try, and it sadly results in a lot of regressions: https://github.com/llvm/llvm-project/commit/0dee41b1ea16a022495a06680410090a25b5bb66
I haven't looked into it yet, but I'm definitely missing something
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159190/new/
https://reviews.llvm.org/D159190
More information about the llvm-commits
mailing list