[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