[PATCH] D107180: [InstCombine] Prevent unnecessary sinks caused by the Freeze

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 30 10:57:46 PDT 2021


lebedev.ri added a comment.

Yeah, i think there's already a profitability check there, so i don't see why we'd want this.
It could be interesting to enhance the sinking to handle multiple uses (see `findCommonDominator()` in `SimplifyIndVar.cpp`),
not sure what compile-time implications that would have.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107180



More information about the llvm-commits mailing list