[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