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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 14:27:48 PDT 2017


efriedma 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) {
----------------
The isIntegerTy(1) check seems unnecessary.


https://reviews.llvm.org/D35856





More information about the llvm-commits mailing list