From a8bb13f989a68030a30181823c58aab6e9c1cc0a Mon Sep 17 00:00:00 2001 From: Bill Wendling Date: Tue, 2 Dec 2008 05:09:00 +0000 Subject: [PATCH] Improve comment. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60398 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/InstructionCombining.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 109e76a47a3..7ad75d24b48 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -4437,14 +4437,13 @@ Instruction *InstCombiner::FoldOrOfICmps(Instruction &I, /// FoldOrWithConstants - This helper function folds: /// -/// ((A | B) & 1) | (B & -2) +/// ((A | B) & C1) | (B & C2) /// /// into: /// -/// (A & 1) | B +/// (A & C1) | B /// -/// The constants aren't important. Only that they don't overlap. (I.e., the XOR -/// of the two constants is "all ones".) +/// when the XOR of the two constants is "all ones" (-1). Instruction *InstCombiner::FoldOrWithConstants(BinaryOperator &I, Value *Op, Value *A, Value *B, Value *C) { ConstantInt *CI1 = dyn_cast(C); -- 2.34.1