[PATCH] D112170: [indvars] Visit all IV users in simplifyAndExtend
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 21 09:51:12 PDT 2021
reames planned changes to this revision.
reames added a comment.
In D112170#3076258 <https://reviews.llvm.org/D112170#3076258>, @nikic wrote:
> The impact is pretty substantial.
Crud. That's definitely too high for the minor code reduction benefit.
I've got one more idea on how to do coarser invalidation once after simplifyAndExtend, let me iterate on this and get a new version up. If that doesn't work, we'll have to simply drop the idea entirely.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112170/new/
https://reviews.llvm.org/D112170
More information about the llvm-commits
mailing list