[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