Page MenuHomePhabricator
Differential D115757

[SLP]Generalize cost model.
Needs ReviewPublic
Actions

Authored by ABataev on Dec 14 2021, 1:13 PM.

Details

Reviewers
RKSimon
anton-afanasyev
dtemirbulatov
Summary

Generalized the cost model estimation. Improved cost model estimation
for repeated scalars (no need to count their cost anymore), improved

cost model for extractelement instructions.

cpu2017

511.povray_r             0.57
520.omnetpp_r           -0.98
521.wrf_r               -0.01
525.x264_r               3.59 <+
526.blender_r           -0.12
531.deepsjeng_r         -0.07
538.imagick_r           -1.42

Geometric mean: 0.21

Diff Detail

Unit TestsFailed
View All

TimeTest
60,050 msx64 debian > Polly - isl unit tests.Polly - isl unit tests::isl_test.sh
Script: -- : 'RUN: at line 1'; polly-isl-test
60,040 msx64 debian > libFuzzer.libFuzzer::large.test
Script: -- : 'RUN: at line 3'; /var/lib/buildkite-agent/builds/llvm-project/build/./bin/clang --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/var/lib/buildkite-agent/builds/llvm-project/compiler-rt/lib/fuzzer -m64 /var/lib/buildkite-agent/builds/llvm-project/compiler-rt/test/fuzzer/LargeTest.cpp -o /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/fuzzer/X86_64DefaultLinuxConfig/Output/large.test.tmp-LargeTest

Event Timeline

ABataev created this revision.Dec 14 2021, 1:13 PM
Herald added a subscriber: hiraditya. · View Herald TranscriptDec 14 2021, 1:13 PM
ABataev requested review of this revision.Dec 14 2021, 1:13 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2021, 1:13 PM
Harbormaster completed remote builds in B139296: Diff 394371.Dec 14 2021, 2:38 PM
ABataev updated this revision to Diff 427351.May 5 2022, 9:07 AM

Rebase

Herald added a project: Restricted Project. · View Herald TranscriptMay 5 2022, 9:07 AM
Herald added a subscriber: vporpo. · View Herald Transcript
Harbormaster completed remote builds in B162935: Diff 427351.May 5 2022, 10:19 AM

Revision Contents
Changeset List

PathSize
llvm/
lib/
Transforms/
Vectorize/
654 lines
test/
Transforms/
SLPVectorizer/
AArch64/
2 lines
35 lines
X86/
18 lines
7 lines
138 lines
138 lines
31 lines
40 lines

Diff 427351

View Options

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/AArch64/horizontal.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/AArch64/vectorizable-selects-min-max.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/PR32086.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/pr47623.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/pr47629-inseltpoison.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/pr47629.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/reduction-same-vals.ll

Loading...
View Options

llvm/test/Transforms/SLPVectorizer/X86/vectorize-reorder-reuse.ll

Loading...