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

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 14:40:15 PDT 2017

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.


More information about the llvm-commits mailing list