[PATCH] D34200: [PM/unswitch] Teach SimpleLoopUnswitch to do non-trivial unswitching, making it no longer even remotely simple.

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 10:00:56 PST 2017


sanjoy accepted this revision.
sanjoy added a comment.
This revision is now accepted and ready to land.

I skimmed through the changes, and they lgtm.  I didn't try to follow the logic closely since I assumed most of the actual mechanics stay the same (but please let me know if that's not true and there are parts of the change that I should review again).

Also, looks like the issue of unstructured control flow was not handled (unswitching a loop with unstructured control flow can create child loops that weren't present before)?  That's fine to address in a separate change.


https://reviews.llvm.org/D34200





More information about the llvm-commits mailing list