[PATCH] D82588: [DSE] Look through memory PHI arguments when removing noop stores in MSSA.

[Zoe Carver via Phabricator via llvm-commits]

zoecarver added a comment.

After running this on the LLVM test suite here are the stats:

**dse.NumRemainingStores**

  Program                                        stats_opt stats_master diff
  
  test-suite...lications/sqlite3/sqlite3.test   13683.00  13723.00      0.3%
  test-suite...decode/alacconvert-decode.test   547.00    547.00        0.0%
  test-suite...nia/pathfinder/pathfinder.test    16.00     16.00        0.0%
  test-suite...s-C/unix-smail/unix-smail.test   218.00    218.00        0.0%
  test-suite...langs-C/unix-tbl/unix-tbl.test   227.00    227.00        0.0%
  test-suite...s/Ptrdist/anagram/anagram.test   124.00    124.00        0.0%
  test-suite.../Benchmarks/Ptrdist/bc/bc.test   983.00    983.00        0.0%
  test-suite.../Benchmarks/Ptrdist/ft/ft.test   154.00    154.00        0.0%
  test-suite.../Benchmarks/Ptrdist/ks/ks.test    88.00     88.00        0.0%
  test-suite...marks/Ptrdist/yacr2/yacr2.test   307.00    307.00        0.0%
  test-suite...Rodinia/backprop/backprop.test    76.00     76.00        0.0%
  test-suite...s/Rodinia/hotspot/hotspot.test    22.00     22.00        0.0%
  test-suite...chmarks/Rodinia/srad/srad.test    50.00     50.00        0.0%
  test-suite.../Prolangs-C/loader/loader.test    93.00     93.00        0.0%
  test-suite...marks/SciMark2-C/scimark2.test    91.00     91.00        0.0%
  Geomean difference                                                    nan%

**dse.NumNoopStores**

  Program                                        stats_opt stats_master diff
  
  test-suite...lications/ClamAV/clamscan.test     2.00      2.00        0.0%
  test-suite...pplications/oggenc/oggenc.test     2.00      2.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...decode/alacconvert-decode.test    NaN       NaN          nan%
  test-suite...encode/alacconvert-encode.test    NaN       NaN          nan%
  test-suite...ce/Applications/Burg/burg.test    NaN       NaN          nan%
  test-suite...ications/JM/ldecod/ldecod.test    NaN       NaN          nan%
  test-suite...ications/JM/lencod/lencod.test    NaN       NaN          nan%
  test-suite...lications/SIBsim4/SIBsim4.test    NaN       NaN          nan%
  test-suite.../Applications/SPASS/SPASS.test    NaN        1.00        nan%
  test-suite...urce/Applications/aha/aha.test    NaN       NaN          nan%
  test-suite...plications/d/make_dparser.test    NaN       NaN          nan%
  test-suite...urce/Applications/hbd/hbd.test    NaN       NaN          nan%
  test-suite...cations/hexxagon/hexxagon.test    NaN       NaN          nan%
  Geomean difference                                                    nan%


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82588