[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 mailing list