[PATCH] D127115: [RFC][DAGCombine] Make sure combined nodes are added back to the worklist in topological order.

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 20 16:59:09 PDT 2023


deadalnix added a comment.

In D127115#4358686 <https://reviews.llvm.org/D127115#4358686>, @RKSimon wrote:

> Yes, I think this is very close now - please can you get some test-suite numbers to highlight any perf differences?

I ran the test suite with and without the patch. The perf difference is well within the measurement noise if there is one at all.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115



More information about the llvm-commits mailing list