[PATCH] D132408: [SimplifyCFG] accumulate bonus insts cost

Thu Sep 8 13:53:28 PDT 2022

yaxunl added a comment.

In D132408#3774420 <https://reviews.llvm.org/D132408#3774420>, @ronlieb wrote:

> Helping our author here: overnight,   i ran cpu 2017 int fp  speed rather than rate, with the patch and without. would that suffice ? 
> sharing summar of results, or if  you really want rate, i can run the rate.  
> the speed fp was .34% faster on GeoMean, and speed int was 0.21% slower.
> 605.mcf 631.deepsjeng were above the 1.75% threshold 
> 605.mcf -3.41%
> 631.deepsjeng 2.32%

The following is the detailed results:

  fp speed        before  after           rerun 3 before  rerun 3 after 
  603.bwaves_s    525.95  527.75  0.34%     
  607.cactuBSSN_s 242.86  246.16  1.36%     
  619.lbm_s       67.54   67.55   0.01%     
  621.wrf_s       88.79   89.74   1.07%     
  627.cam4_s      124.71  123.57  -0.91%      
  628.pop2_s      52.73   53.06   0.63%     
  638.imagick_s   238.67  240.62  0.82%     
  644.nab_s       370.06  367.73  -0.63%      
  649.fotonik3d_s 82.96   83.76   0.96%     
  654.roms_s      268.70  268.22  -0.18%      
  geomean         158.36  158.90  0.34%     

  int speed             
  600.perlbench_s 4.34  4.32  -0.46%      
  602.gcc_s       8.58  8.33  -2.91%    8.51  8.51  0.00%
  605.mcf_s       6.73  6.50  -3.42%    6.74  6.51  -3.41%
  620.omnetpp_s   3.53  3.46  -1.98%    3.54  3.51  -0.85%
  623.xalancbmk_s 8.24  8.22  -0.24%      
  625.x264_s      9.97  9.96  -0.10%      
  631.deepsjeng_s 3.06  3.26  6.54%     3.02  3.09  2.32%
  641.leela_s     3.79  3.83  1.06%     
  648.exchange2_s 7.37  7.43  0.81%     
  657.xz_s        19.65 19.45 -1.02%      
  geomean         6.44  6.42  -0.21%      

@fhahn Is this acceptable? Thanks.

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

https://reviews.llvm.org/D132408