[PATCH] D99774: [LoopUtils] Populate sibling loops in reverse program order on new pass manager

[JinGu Kang via Phabricator via llvm-commits]

jaykang10 added a comment.

In D99774#2704838 <https://reviews.llvm.org/D99774#2704838>, @fhahn wrote:

> Could you share any data of the impact this change has on runtime-performance/code-size on benchmarks?

I have checked exec_time/size with llvm-test-suite on x86 and the result is as below.

  Tests: 2892
  Metric: exec_time
  
  /usr/local/lib/python2.7/dist-packages/scipy/stats/stats.py:308: RuntimeWarning: divide by zero encountered in log
    log_a = np.log(np.array(a, dtype=dtype))
  Program                                        base   new    diff 
   test-suite...te/GCC-C-execute-980505-2.test     0.00   0.00  inf%
   test-suite...ute/GCC-C-execute-pr49768.test     0.00   0.00  inf%
   test-suite...cute/GCC-C-execute-simd-6.test     0.00   0.00  inf%
   test-suite...ute/GCC-C-execute-pr68328.test     0.00   0.00  inf%
   test-suite...d_ops_test_op_paddusw_152.test     0.00   0.00  inf%
   test-suite...te/GCC-C-execute-stkalign.test     0.00   0.00  inf%
   test-suite.../GCC-C-execute-20071216-1.test     0.00   0.00  inf%
   test-suite...ute/GCC-C-execute-pr47538.test     0.00   0.00  inf%
   test-suite...ecute/GCC-C-execute-mod-1.test     0.00   0.00  inf%
   test-suite...ute/GCC-C-execute-pr46316.test     0.00   0.00  inf%
   test-suite...md_ops_test_op_pmaxsw_177.test     0.00   0.00  inf%
   test-suite...execute-struct-aliasing-1.test     0.00   0.00  inf%
   test-suite...cute/GCC-C-execute-enum-1.test     0.00   0.00  inf%
   test-suite.../GCC-C-execute-20010114-1.test     0.00   0.00  inf%
   test-suite...ute/GCC-C-execute-pr46309.test     0.00   0.00  inf%
   Geomean difference                                           nan%
                  base            new         diff
  count  2876.000000    2876.000000    2874.000000
  mean   485.557850     456.966905     inf        
  std    13703.055405   12752.463246  NaN         
  min    0.000000       0.000000      -1.000000   
  25%    0.000400       0.000400      -0.250000   
  50%    0.000700       0.000700       0.000000   
  75%    0.010925       0.010925       0.295752   
  max    526452.168000  550562.615000  inf

  Tests: 2892
  Metric: size
  
  Program                                        base    new     diff 
   test-suite...ute/GCC-C-execute-pr41917.test   8016.00 8024.00  0.1%
   test-suite...ute/GCC-C-execute-pr60072.test   8016.00 8024.00  0.1%
   test-suite...ute/GCC-C-execute-pr47155.test   8040.00 8048.00  0.1%
   test-suite...ute/GCC-C-execute-pr82387.test   8048.00 8056.00  0.1%
   test-suite.../GCC-C-execute-20001009-2.test   8048.00 8056.00  0.1%
   test-suite.../GCC-C-execute-20041201-1.test   8056.00 8064.00  0.1%
   test-suite...ute/GCC-C-execute-pr57321.test   8072.00 8080.00  0.1%
   test-suite...ute/GCC-C-execute-pr58385.test   8072.00 8080.00  0.1%
   test-suite...cute/GCC-C-execute-pure-1.test   8080.00 8088.00  0.1%
   test-suite...ute/GCC-C-execute-pr81503.test   8096.00 8104.00  0.1%
   test-suite...ute/GCC-C-execute-pr48717.test   8104.00 8112.00  0.1%
   test-suite...ute/GCC-C-execute-pr65216.test   8120.00 8128.00  0.1%
   test-suite.../GCC-C-execute-20040302-1.test   8136.00 8144.00  0.1%
   test-suite...e/GCC-C-execute-pr52979-1.test   8176.00 8184.00  0.1%
   test-suite...e/GCC-C-execute-pr52979-2.test   8176.00 8184.00  0.1%
   Geomean difference                                             nan%
                 base           new         diff
  count  2.484000e+03  2.484000e+03  2484.000000
  mean   3.340861e+04  3.340918e+04  0.000047   
  std    1.803869e+05  1.803872e+05  0.000201   
  min    7.992000e+03  7.992000e+03  0.000000   
  25%    8.264000e+03  8.264000e+03  0.000000   
  50%    8.344000e+03  8.344000e+03  0.000000   
  75%    1.742200e+04  1.742200e+04  0.000000   
  max    7.648424e+06  7.648432e+06  0.000998


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

https://reviews.llvm.org/D99774