[PATCH] D25200: [InstCombine] New opportunities for FoldAndOfICmp and FoldXorOfICmp
David Majnemer via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 28 13:09:10 PDT 2016
majnemer added inline comments.
================
Comment at: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:760
+ BitGroupCheck BGC;
+ auto Inst = dyn_cast<Instruction>(LHS);
+
----------------
`auto *`
================
Comment at: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:762
+
+ if (Inst && Inst->getOpcode() != Instruction::And)
+ return BGC;
----------------
Er, shouldn't this be `!Inst || Inst->getOpcode() != Instruction::And`
https://reviews.llvm.org/D25200
More information about the llvm-commits
mailing list