[PATCH] D28190: [LVI] Switch from BFS to DFS exploration strategy
Daniel Berlin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 2 11:08:00 PST 2017
dberlin accepted this revision.
dberlin added a comment.
This revision is now accepted and ready to land.
It would be nice if you had a way to test that this changes nothing. If anything, it actually should improve results, since the optimal lattice values for loops and irreducible control flow may only be achievable with proper iteration order.
My usual plan is to save-temps llvm into bc files, and then generate results before and after.
Assuming you do *something* to verify we aren't getting worse answers, i think this can go in.
More information about the llvm-commits