[PATCH] D77325: [InstCombine] Don't limit operands in eraseInstFromFunction()
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 4 07:24:18 PDT 2020
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
This blames all the way back to rL80483 <https://reviews.llvm.org/rL80483>:
> inline the trivial AddToWorkList/RemoveFromWorkList methods into their callers. simplify ReplaceInstUsesWith. Make EraseInstFromFunction only add operands to the worklist if there aren't too many of them (this was a scalability win for crazy programs that was only infrequently enforced). Switch more code to using EraseInstFromFunction instead of duplicating it inline. Change some fcmp/icmp optimizations to modify fcmp/icmp in place instead of creating a new one and deleting the old one just to change the predicate.
So much like waymarking, some optimization that doesn't have clear consequences.
I think this shouldn't be here, at least for the consistency reasons, unless there's a testcase that shows otherwise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77325/new/
https://reviews.llvm.org/D77325
More information about the llvm-commits
mailing list