[PATCH] D79391: [DSE] Remove noop stores in MSSA.

Zoe Carver via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 20 12:38:01 PDT 2020


zoecarver added a comment.

> Yes, when you build the test-suite, you can set TEST_SUITE_COLLECT_STATS=On to collect stats.

Thanks that's super helpful.

-----

Here are the results:

**Redundant Stores:** (no change)

  Metric: dse.NumRedundantStores
  
  Program                                        stats_opt stats_master diff
  
  test-suite...lications/ClamAV/clamscan.test     2.00      2.00        0.0%
  test-suite...ications/JM/ldecod/ldecod.test     3.00      3.00        0.0%
  test-suite...ications/JM/lencod/lencod.test     4.00      4.00        0.0%
  test-suite.../Applications/SPASS/SPASS.test     2.00      2.00        0.0%
  test-suite.../Applications/lemon/lemon.test     2.00      2.00        0.0%
  test-suite...pplications/oggenc/oggenc.test     8.00      8.00        0.0%
  test-suite...pplications/treecc/treecc.test     4.00      4.00        0.0%
  test-suite...marks/7zip/7zip-benchmark.test     6.00      6.00        0.0%
  test-suite...ProxyApps-C++/CLAMR/CLAMR.test     1.00      1.00        0.0%
  test-suite...hmarks/McCat/15-trie/trie.test     1.00      1.00        0.0%
  test-suite...nsumer-lame/consumer-lame.test     3.00      3.00        0.0%
  test-suite...TimberWolfMC/timberwolfmc.test     1.00      1.00        0.0%
  test-suite.../Prolangs-C/loader/loader.test     1.00      1.00        0.0%
  test-suite.../Benchmarks/Ptrdist/bc/bc.test     1.00      1.00        0.0%
  test-suite...arks/mafft/pairlocalalign.test     6.00      6.00        0.0%
  Geomean difference                                                    nan%
         stats_opt  stats_master  diff
  count  15.000000  15.000000     15.0
  mean   3.000000   3.000000      0.0
  std    2.203893   2.203893      0.0
  min    1.000000   1.000000      0.0
  25%    1.000000   1.000000      0.0
  50%    2.000000   2.000000      0.0
  75%    4.000000   4.000000      0.0
  max    8.000000   8.000000      0.0

**Execution Time:** (slight improvement)

  Metric: exec_time
  
  Program                                        stats_opt stats_master diff
  
  test-suite...urce/Applications/hbd/hbd.test     0.00      0.00       -28.1%
  test-suite...adpcm/rawcaudio/rawcaudio.test     0.00      0.00       -23.1%
  test-suite...itBench/uudecode/uudecode.test     0.10      0.12       20.6%
  test-suite...urce/Applications/aha/aha.test     1.26      1.02       -19.1%
  test-suite.../Benchmarks/Ptrdist/bc/bc.test     0.69      0.83       18.9%
  test-suite...flt/LoopRestructuring-flt.test     4.32      3.57       -17.4%
  test-suite...count/automotive-bitcount.test     0.09      0.08       -16.9%
  test-suite...ks/McCat/12-IOtest/iotest.test     0.28      0.33       16.4%
  test-suite.../Benchmarks/Ptrdist/ft/ft.test     0.88      1.01       14.7%
  test-suite...chmarks/MallocBench/gs/gs.test     0.04      0.05       14.6%
  test-suite...arching-dbl/Searching-dbl.test     4.16      3.55       -14.6%
  test-suite...ve-susan/automotive-susan.test     0.04      0.04       -14.4%
  test-suite...t/StatementReordering-flt.test     3.36      2.89       -14.0%
  test-suite...mbolics-flt/Symbolics-flt.test     1.26      1.09       -13.8%
  test-suite...mbolics-dbl/Symbolics-dbl.test     3.16      2.73       -13.7%
  Geomean difference                                                   -1.4%
          stats_opt  stats_master        diff
  count  198.000000  198.000000    198.000000
  mean   2.136898    2.102667     -0.010418
  std    4.097594    4.069014      0.079837
  min    0.000800    0.000900     -0.281250
  25%    0.036075    0.035250     -0.068807
  50%    0.616700    0.650100      0.000000
  75%    3.317450    3.089325      0.045511
  max    32.665900   31.948900     0.205853

**Execution Time (filter short):** (mostly improved)

  Tests: 198
  Short Running: 96 (filtered out)
  Remaining: 102
  Metric: exec_time
  
  Program                                        stats_opt stats_master diff
  
  test-suite...urce/Applications/aha/aha.test     1.26      1.02       -19.1%
  test-suite.../Benchmarks/Ptrdist/bc/bc.test     0.69      0.83       18.9%
  test-suite...flt/LoopRestructuring-flt.test     4.32      3.57       -17.4%
  test-suite.../Benchmarks/Ptrdist/ft/ft.test     0.88      1.01       14.7%
  test-suite...arching-dbl/Searching-dbl.test     4.16      3.55       -14.6%
  test-suite...t/StatementReordering-flt.test     3.36      2.89       -14.0%
  test-suite...mbolics-flt/Symbolics-flt.test     1.26      1.09       -13.8%
  test-suite...mbolics-dbl/Symbolics-dbl.test     3.16      2.73       -13.7%
  test-suite...C/Packing-flt/Packing-flt.test     4.71      4.10       -13.0%
  test-suite...ctions-dbl/Reductions-dbl.test     4.06      3.54       -12.7%
  test-suite...rimaran/enc-3des/enc-3des.test     1.90      1.66       -12.6%
  test-suite.../Trimaran/enc-md5/enc-md5.test     1.69      1.48       -12.4%
  test-suite...yApps-C++/PENNANT/PENNANT.test     0.62      0.70       12.3%
  test-suite...arching-flt/Searching-flt.test     3.98      3.49       -12.3%
  test-suite...s/Ptrdist/anagram/anagram.test     0.64      0.72       12.2%
  Geomean difference                                                    nan%
          stats_opt  stats_master        diff
  count  102.000000  101.000000    101.000000
  mean   4.070663    4.036584     -0.015677
  std    4.994630    4.988885      0.080786
  min    0.604600    0.635000     -0.191148
  25%    1.660050    1.549400     -0.083075
  50%    3.237450    3.072300     -0.005657
  75%    4.359050    4.313500      0.046250
  max    32.665900   31.948900     0.189496

**Compile Time:** (unexpectedly improved but, that might just be noise)

  Metric: compile_time
  
  Program                                        stats_opt stats_master diff
  
  test-suite...rks/tramp3d-v4/tramp3d-v4.test    45.98     52.93       15.1%
  test-suite.../Trimaran/enc-pc1/enc-pc1.test     0.18      0.20        9.6%
  test-suite...netbench-crc/netbench-crc.test     0.20      0.21        9.0%
  test-suite...hmarks/VersaBench/bmm/bmm.test     0.14      0.15        8.4%
  test-suite...marks/SciMark2-C/scimark2.test     0.94      1.02        8.1%
  test-suite.../Benchmarks/Ptrdist/ft/ft.test     0.57      0.61        7.6%
  test-suite...marks/7zip/7zip-benchmark.test   122.38    131.64        7.6%
  test-suite.../Trimaran/enc-rc4/enc-rc4.test     0.13      0.14        7.5%
  test-suite.../Trimaran/enc-md5/enc-md5.test     0.29      0.31        7.5%
  test-suite...ngs-C/fixoutput/fixoutput.test     0.16      0.18        7.2%
  test-suite...netbench-url/netbench-url.test     0.50      0.53        7.1%
  test-suite.../Prolangs-C/loader/loader.test     0.73      0.78        7.1%
  test-suite...arks/VersaBench/dbms/dbms.test     2.30      2.47        7.0%
  test-suite...langs-C/allroots/allroots.test     0.20      0.22        7.0%
  test-suite...ngs-C/assembler/assembler.test     1.38      1.48        6.9%
  Geomean difference                                                    0.3%
          stats_opt  stats_master        diff
  count  198.000000  198.000000    198.000000
  mean   5.392878    5.432042      0.003577
  std    13.798743   14.112571     0.039915
  min    0.053700    0.052200     -0.067272
  25%    0.318475    0.314125     -0.027627
  50%    1.606700    1.594650     -0.000062
  75%    3.390850    3.419850      0.031306
  max    122.376700  131.644500    0.151016


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79391





More information about the llvm-commits mailing list