From 61724e6c893947b936f39cf5e286443cfcc94b98 Mon Sep 17 00:00:00 2001 From: Sanjay Patel Date: Thu, 29 Sep 2016 17:48:19 +0000 Subject: [PATCH] fix formatting; NFC git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@282737 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../InstCombine/InstCombineSelect.cpp | 20 +++++++++---------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/lib/Transforms/InstCombine/InstCombineSelect.cpp b/lib/Transforms/InstCombine/InstCombineSelect.cpp index a18b33475b2..5af0dab940d 100644 --- a/lib/Transforms/InstCombine/InstCombineSelect.cpp +++ b/lib/Transforms/InstCombine/InstCombineSelect.cpp @@ -920,28 +920,26 @@ static Instruction *foldAddSubSelect(SelectInst &SI, /// we may be able to create an i1 select which can be further folded to /// logical ops. static Instruction *foldSelectExtConst(InstCombiner::BuilderTy &Builder, - SelectInst &SI, Instruction *EI, - const APInt &C, bool isExtTrueVal, - bool isSigned) { - Value *SmallVal = EI->getOperand(0); + SelectInst &SI, Instruction *ExtInst, + const APInt &C, bool IsExtTrueVal, + bool IsSigned) { + Value *SmallVal = ExtInst->getOperand(0); Type *SmallType = SmallVal->getType(); - // TODO Handle larger types as well? Note this requires adjusting - // FoldOpIntoSelect as well. + // TODO: Handle larger types? That requires adjusting FoldOpIntoSelect too. if (!SmallType->getScalarType()->isIntegerTy(1)) return nullptr; - if (C != 0 && (isSigned || C != 1) && - (!isSigned || !C.isAllOnesValue())) + if (C != 0 && (IsSigned || C != 1) && (!IsSigned || !C.isAllOnesValue())) return nullptr; Value *SmallConst = ConstantInt::get(SmallType, C.trunc(1)); - Value *TrueVal = isExtTrueVal ? SmallVal : SmallConst; - Value *FalseVal = isExtTrueVal ? SmallConst : SmallVal; + Value *TrueVal = IsExtTrueVal ? SmallVal : SmallConst; + Value *FalseVal = IsExtTrueVal ? SmallConst : SmallVal; Value *Select = Builder.CreateSelect(SI.getOperand(0), TrueVal, FalseVal, "fold." + SI.getName(), &SI); - if (isSigned) + if (IsSigned) return new SExtInst(Select, SI.getType()); return new ZExtInst(Select, SI.getType());