[PATCH] D35856: [InlineCost] Simplify more 'and' and 'or' operations.

[Chad Rosier via Phabricator via llvm-commits]

mcrosier marked an inline comment as done.
mcrosier added inline comments.


================
Comment at: lib/Analysis/InlineCost.cpp:811
+  // generic simplify instruction only works if both operands are constants.
+  if (I.getType()->isIntegerTy(1)) {
+    for (unsigned i = 0; i < 2; ++i) {
----------------
efriedma wrote:
> The isIntegerTy(1) check seems unnecessary.
Yup!  I've uploaded a new patch.


https://reviews.llvm.org/D35856