[llvm] ConstraintElim: add dry-run routine to fail early (PR #99670)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 29 11:02:54 PDT 2024


artagnon wrote:

Tweaking the patch was pretty straight-forward actually. There are no binary changes on SPEC2017 after tweaking the factor to 1.6. Compile time improvement is 0.6%. As I mentioned, there is no way to absolutely guarantee that there won't be binary changes in a theoretical degenerate case, but I think the trade-off I've chosen is practical, and will allow us to move forward.

```
Program                                       compile_time
                                              baseline     patch    diff
External/S...96.specrand_fs/996.specrand_fs       1.62         1.72   6.0%
External/S...99.specrand_ir/999.specrand_ir       1.67         1.76   5.6%
External/S...P2017speed/644.nab_s/644.nab_s     102.36       108.04   5.5%
External/S...NT2017rate/505.mcf_r/505.mcf_r      22.42        23.25   3.7%
External/S...T2017speed/605.mcf_s/605.mcf_s      22.29        22.98   3.1%
External/S...017speed/625.x264_s/625.x264_s     365.08       373.04   2.2%
External/S...te/520.omnetpp_r/520.omnetpp_r    3009.42      3027.51   0.6%
External/S...17rate/541.leela_r/541.leela_r     355.39       357.28   0.5%
External/S...te/526.blender_r/526.blender_r    7542.34      7568.07   0.3%
External/S...rate/510.parest_r/510.parest_r   10715.75     10749.72   0.3%
External/S...rate/511.povray_r/511.povray_r     802.93       805.45   0.3%
External/S...ed/638.imagick_s/638.imagick_s    1035.18      1038.16   0.3%
External/S...ed/620.omnetpp_s/620.omnetpp_s    3026.72      3032.78   0.2%
External/S...23.xalancbmk_r/523.xalancbmk_r    6407.85      6414.59   0.1%
Bitcode/Be...hmarks/Halide/blur/halide_blur       1.43         1.43   0.0%
External/S...T2017speed/602.gcc_s/602.gcc_s    6570.48      6569.85  -0.0%
External/S...23.xalancbmk_s/623.xalancbmk_s    6397.61      6396.53  -0.0%
External/S...INT2017speed/657.xz_s/657.xz_s     129.01       128.97  -0.0%
External/S...2017rate/525.x264_r/525.x264_r     369.01       368.63  -0.1%
External/S...00.perlbench_r/500.perlbench_r    1074.48      1071.83  -0.2%
External/S...NT2017rate/502.gcc_r/502.gcc_r    6543.43      6523.43  -0.3%
External/S...te/538.imagick_r/538.imagick_r    1136.65      1132.24  -0.4%
External/S...2017rate/508.namd_r/508.namd_r     539.89       537.10  -0.5%
External/S...7speed/641.leela_s/641.leela_s     353.56       350.23  -0.9%
External/S...98.specrand_is/998.specrand_is       1.77         1.75  -1.0%
External/S...CINT2017rate/557.xz_r/557.xz_r     131.92       129.86  -1.6%
External/S...00.perlbench_s/600.perlbench_s    1194.66      1174.82  -1.7%
External/S...FP2017rate/544.nab_r/544.nab_r     113.08       111.06  -1.8%
External/S...31.deepsjeng_s/631.deepsjeng_s      76.04        74.23  -2.4%
External/S...31.deepsjeng_r/531.deepsjeng_r      76.10        71.79  -5.7%
External/S...FP2017rate/519.lbm_r/519.lbm_r       8.87         8.23  -7.2%
External/S...97.specrand_fr/997.specrand_fr       1.65         1.53  -7.3%
External/S...P2017speed/619.lbm_s/619.lbm_s       9.61         8.05 -16.3%
                           Geomean difference                        -0.6%
```

https://github.com/llvm/llvm-project/pull/99670


More information about the llvm-commits mailing list