[PATCH] D124526: [SimpleLoopUnswitch] Collect either logical ANDs/ORs but not both.

[Florian Hahn via Phabricator via llvm-commits]

fhahn updated this revision to Diff 426014.
fhahn marked 2 inline comments as done.
fhahn added a comment.

Update with alternative approach where helper functions are used to match either AND/OR but skipping (select _, true, false). This also removes a piece of code that may modify the function even if no unswitching happens (and we would claim no changes have been made)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124526/new/

https://reviews.llvm.org/D124526

Files:
  llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-freeze.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-invariant-select-bug.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch.ll
  llvm/test/Transforms/SimpleLoopUnswitch/trivial-unswitch-freeze-individual-conditions.ll
  llvm/test/Transforms/SimpleLoopUnswitch/trivial-unswitch-logical-and-or.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124526.426014.patch
Type: text/x-patch
Size: 18229 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220429/d91456e9/attachment.bin>