[PATCH] D29220: [InstCombine] Re-enable guard canonicalization for AND and NOT OR

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 03:24:06 PST 2017


mkazantsev created this revision.

The following changes:

r293061: "[InstCombine] Canonicalize guards for NOT OR condition"
r293058: "[InstCombine] Canonicalize guards for AND condition"

were reverted by the change

https://reviews.llvm.org/rL293227: "Revert a couple of InstCombine/Guard checkins"

because they contained a bug related to handling guards with multiple
arguments. The problem was that the extra arguments of the initial
guard were not passed to the newly-created guards.

This patch re-enables canonicalization of guards for AND and NOT OR
conditions and adds proper tests for them.


https://reviews.llvm.org/D29220

Files:
  lib/Transforms/InstCombine/InstCombineCalls.cpp
  test/Transforms/InstCombine/call-guard.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29220.86035.patch
Type: text/x-patch
Size: 4842 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170127/d21f432f/attachment.bin>


More information about the llvm-commits mailing list