[PATCH] D35182: [InstCombine] remove one-use restriction for not (cmp P, A, B) --> cmp P', A, B

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D35182#828364, @craig.topper wrote:

> That's not really the IR for that C code is it. The C code has two global variables and the IR has two arguments.


Oops - that's correct. I simplified the example from PR27431 to:

  int fn1(int a, char b) { return (a && b) || (a && b); }

It shows the same problem though AFAICT.


https://reviews.llvm.org/D35182