[PATCH] D65148: [SimplifyCFG] Bump phi-node-folding-threshold from 2 to 3

Sat Sep 7 03:54:03 PDT 2019

xbolva00 added a comment.

I compared your jsons using

  python3 compare.py results-testsuite-old.json results-testsuite-new.json --filter-short

  Tests: 1318
  Short Running: 736 (filtered out)
  Remaining: 582
  Metric: exec_time

  Program                                        results-testsuite-old results-testsuite-new diff  
   test-suite...C++/Shootout-C++-heapsort.test     3.49                 10.84                210.7%
   test-suite...e/Benchmarks/Misc/flops-8.test     1.25                  3.70                195.3%
   test-suite...out-C++/Shootout-C++-hash.test     0.77                  1.97                155.1%
   test-suite...C/Packing-dbl/Packing-dbl.test     3.56                  8.12                128.1%
   test-suite...est:BM_GEN_LIN_RECUR_RAW/44217   339.78                737.62                117.1%
   test-suite...test:BM_PLANCKIAN_LAMBDA/44217   984.23                2096.79               113.0%
   test-suite...s/Rodinia/hotspot/hotspot.test     0.62                  1.25                102.1%
   test-suite...st:BM_MemCmp<64, EqZero, Last>   10153.35              584.76                -94.2%
   test-suite...est:BM_MemCmp<64, EqZero, Mid>   9845.54               581.01                -94.1%
   test-suite...mCmp<7, GreaterThanZero, None>   2168.74               4178.58               92.7% 
   test-suite...d-warshall/floyd-warshall.test     8.15                 15.46                89.6% 
   test-suite...t:BM_MemCmp<64, EqZero, First>   2859.27               401.51                -86.0%
   test-suite...ests/Vector/Vector-build2.test     9.43                  1.91                -79.7%
   test-suite...algebra/kernels/syrk/syrk.test    24.44                  5.37                -78.0%
   test-suite...mCmp<15, GreaterThanZero, Mid>   1615.43               2841.94               75.9% 

Any my mistake? Because these perf results do not look so great.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65148