[PATCH] D48851: [PM/LoopUnswitch] Fix the first and immediate components of PR37889.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 6 17:58:55 PDT 2018


chandlerc updated this revision to Diff 154476.
chandlerc added a comment.

Update with the bug found by Fedor fixed and test cases nicely enhanced to
cover all of this stuff.

I've also added some asserts to document that other possible updates to the
loop structure aren't needed. The code here is pretty trivial and the rest of
the patch already got LGTMs so I'm gonna go ahead and submit this.

This also updates to have the extra test requested in review and more comments
in the tests.


Repository:
  rL LLVM

https://reviews.llvm.org/D48851

Files:
  llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch.ll
  llvm/test/Transforms/SimpleLoopUnswitch/trivial-unswitch.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48851.154476.patch
Type: text/x-patch
Size: 36762 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180707/b1d9c9d2/attachment.bin>


More information about the llvm-commits mailing list