[PATCH] D54730: [DomTree] Fix order of domtree updates in MergeBlockIntoPredecessor.

Mon Nov 26 12:48:09 PST 2018

brzycki added a comment.

I just re-ran both to make sure it wasn't caused by other factors on the host. There's some movement in the <= 2% range but tramp3d-v4 is the same regression.

  #  left column: D54730/results.json
  # right column: tip/results.json
  #  metric name: compile_time
       43.7640 -> 39.6280      [ 10.44%]  CTMark/tramp3d-v4/tramp3d-v4.test
       26.4680 <- 27.0880      [  2.34%]  CTMark/sqlite3/sqlite3.test
      130.5600 -> 128.1000     [  1.92%]  CTMark/7zip/7zip-benchmark.test
       35.4600 <- 36.1000      [  1.80%]  CTMark/kimwitu++/kc.test
       32.2400 -> 31.7520      [  1.54%]  CTMark/consumer-typeset/consumer-typeset.test
       23.3600 <- 23.6480      [  1.23%]  CTMark/mafft/pairlocalalign.test
       52.7280 -> 52.5520      [  0.33%]  CTMark/lencod/lencod.test
       50.8120 <- 50.9560      [  0.28%]  CTMark/ClamAV/clamscan.test
       44.1040 <- 44.2160      [  0.25%]  CTMark/SPASS/SPASS.test
       86.8800 -> 86.8200      [  0.07%]  CTMark/Bullet/bullet.test

Repository:
  rL LLVM

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

https://reviews.llvm.org/D54730