[PATCH] D52362: [CloneFunction] Simplify previously unsimplifiable instructions

[Anna Thomas via Phabricator via llvm-commits]

anna added a comment.

Results on CTMark does show negative compile time impact on couple of benchmarks - run on a haswell machine over 4 runs with and without patch:

  Tests: 10
  Metric: compile_time
  
  Program                                         Base    Patch   diff 
                                                                   
  Bullet/bullet                                  59.27  67.88 14.5%
  tramp3d-v4/tramp3d-v4                          56.98  64.87 13.8%
  7zip/7zip-benchmark                            91.63 103.92 13.4%
  kimwitu++/kc                                   31.60  35.39 12.0%
  lencod/lencod                                  41.75  43.57  4.4%
  sqlite3/sqlite3                                40.47  40.02 -1.1%
  SPASS/SPASS                                    34.11  34.33  0.6%
  mafft/pairlocalalign                           23.81  23.71 -0.4%
  consumer-typeset/consumer-typeset              26.28  26.22 -0.2%
  ClamAV/clamscan                                37.51  37.54  0.1%

Maybe we're hitting on the degenerate case - Ran  SimplifyInstruction, but no change in simplification. So future passes still had to iterate on same number of instructions...


Repository:
  rL LLVM

https://reviews.llvm.org/D52362