[PATCH] D150447: [MachineSink] Don't reject sinking because of a dead def in isProfitableToSinkTo()
Alexander Kornienko via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 22 10:21:26 PDT 2023
alexfh added a comment.
As a part of our routine internal release testing we found out that the Ptrdist/yacr2 benchmark <https://github.com/llvm/llvm-test-suite/tree/main/MultiSource/Benchmarks/Ptrdist/yacr2> shows an around 10% regression after this commit on Haswell CPUs. It's not blocking us in any way, but you might be interested in confirming that this doesn't indicate an undesired side effect on the generated code.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150447/new/
https://reviews.llvm.org/D150447
More information about the llvm-commits
mailing list