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

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc updated this revision to Diff 119114.
chandlerc marked an inline comment as done.
chandlerc added a comment.

Update addressing the remaining comments from Sanjoy.

Notably, this includes the majority of work to address the factoring comments.
I've now factored several parts of this into separate functions to make things
a bit more consumable.

I think this is all ready for review again.


https://reviews.llvm.org/D34200

Files:
  llvm/include/llvm/Analysis/LoopInfo.h
  llvm/include/llvm/Analysis/LoopInfoImpl.h
  llvm/include/llvm/Transforms/Scalar/LoopPassManager.h
  llvm/include/llvm/Transforms/Scalar/SimpleLoopUnswitch.h
  llvm/lib/Analysis/LoopPass.cpp
  llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
  llvm/test/Transforms/SimpleLoopUnswitch/2006-06-27-DeadSwitchCase.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-cost.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34200.119114.patch
Type: text/x-patch
Size: 177266 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171016/7a2d5608/attachment-0001.bin>