[PATCH] D99276: [ValueTracking] peek through min/max to find isKnownToBeAPowerOfTwo

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon accepted this revision.
RKSimon added a comment.

LGTM with one minor



================
Comment at: llvm/include/llvm/IR/PatternMatch.h:1738
     // it is the values returned by the select that are being compared.
-    Value *TrueVal = SI->getTrueValue();
-    Value *FalseVal = SI->getFalseValue();
-    Value *LHS = Cmp->getOperand(0);
-    Value *RHS = Cmp->getOperand(1);
+    auto *TrueVal = SI->getTrueValue();
+    auto *FalseVal = SI->getFalseValue();
----------------
lebedev.ri wrote:
> Yeah, i think that's fine..
Really we should be avoiding auto for anything but casts or iterators


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99276/new/

https://reviews.llvm.org/D99276