llvm / llvm/llvm-project

Non-power-of-2 SLP vectorization regressions (and improvements) on RISC-V

Open
#198,485 5 comments 0 reactions 2 assignees Claimed by @bababuck View on GitHub
backend:RISC-V llvm:SLPVectorizer
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

LNT picked up a handful of regressions after the non-power-of-two SLP vectorization change #196825 : http://lnt.llvm.org/db_default/v4/nts/22

Just flagging this here as there's likely some cost modelling issues to be worked through, see

- [SingleSource/Benchmarks/BenchmarkGame/n-body](http://lnt.llvm.org/db_default/v4/nts/22/graph?test.1339=3)
- [MultiSource/Benchmarks/McCat/08-main/main](http://lnt.llvm.org/db_default/v4/nts/22/graph?test.1224=3)
- [MultiSource/Benchmarks/TSVC/LinearDependence-flt/LinearDependence-flt](http://lnt.llvm.org/db_default/v4/nts/22/graph?test.1290=3)

There are also some nice improvements too though, [SingleSource/Benchmarks/Shootout-C++/Shootout-C++-matrix](http://lnt.llvm.org/db_default/v4/nts/22/graph?test.1428=3) is 12% faster

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.