[PATCH] D45734: [reassociate] Fix excessive revisits when processing long chains of reassociatable instructions.

[Daniel Sanders via Phabricator via llvm-commits]

dsanders added a comment.
Herald added a reviewer: javed.absar.

I have some performance figures from the LLVM test-suite now using the --benchmarking-only option. Here's the summaries reported by lnt:

| metric         | geomean before patch | geomean after patch | delta   |
| -------------- | -------------------- | ------------------- | ------- |
| compile time   | 0.1956               | 0.1261              | -35.54% |
| execution time | 0.3240               | 0.3237              | -       |
| code size      | 7365.4459            | 7365.6079           | -       |
|

The results have a few wins and losses on compile-time, mostly in the +/- 2.5% range. There was one outlier though:

| Performance Regressions - compile_time                 | Δ     | Previous | Current |
| MultiSource/Benchmarks/ASC_Sequoia/CrystalMk/CrystalMk | 9.82% | 2.0473   | 2.2483  |


Repository:
  rL LLVM

https://reviews.llvm.org/D45734