[PATCH] D89693: [AArch64] Favor post-increments

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Just curious if there's something in particular you are concerned about? I am just asking because then I can focus on that.

I have rerun experiment and instead of 5 runs I am comparing 10 runs by taking the minimum runtime of each of the runs in this way:

  ../test-suite/utils/compare.py --filter-short before.1.json before.2.json before.3.json before.4.json before.5.json before.6.json before.7.json before.8.json before.9.json before.10.json vs after.1.json after.2.json after.3.json after.4.json after.5.json after.6.json after.7.json after.8.json after.9.json after.10.json

This gives:

  Tests: 10
  Short Running: 4 (filtered out)
  Remaining: 6
  Metric: exec_time
  Program                                        lhs    rhs    diff 
   test-suite...Mark/mafft/pairlocalalign.test    56.41  56.48  0.1%
   test-suite :: CTMark/Bullet/bullet.test         9.99   9.98 -0.0%
   test-suite :: CTMark/SPASS/SPASS.test          25.11  25.08 -0.1%
   test-suite :: CTMark/lencod/lencod.test         8.58   8.57 -0.2%
   test-suite...TMark/7zip/7zip-benchmark.test    10.16  10.11 -0.4%
   test-suite...:: CTMark/sqlite3/sqlite3.test     3.86   3.84 -0.5%
   Geomean difference                                          -0.2%
               lhs        rhs      diff
  count  6.000000   6.000000   6.000000
  mean   19.016133  19.009183 -0.001906
  std    19.665927  19.699874  0.002340
  min    3.858400   3.839700  -0.004847
  25%    8.933775   8.919975  -0.003747
  50%    10.071450  10.047500 -0.001556
  75%    21.368850  21.336775 -0.000573
  max    56.406300  56.476400  0.001243


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

https://reviews.llvm.org/D89693