[PATCH] D125755: [AggressiveInstcombine] Conditionally fold saturated fptosi to llvm.fptosi.sat
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 18 03:40:12 PDT 2022
RKSimon added a comment.
OK - speaking to @spatel offline there was a concern that AIC was becoming a bit of a dumping ground for combines that didn't really fit anywhere else.
It might be that we need to consider a CostDrivenInstCombine pass or something instead? But then I'm not sure how much that will overlap with VectorCombine etc,
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125755/new/
https://reviews.llvm.org/D125755
More information about the llvm-commits
mailing list