diff options
| author | Michael Zolotukhin <mzolotukhin@apple.com> | 2015-10-24 20:16:42 +0000 |
|---|---|---|
| committer | Michael Zolotukhin <mzolotukhin@apple.com> | 2015-10-24 20:16:42 +0000 |
| commit | 1eeb2da7d4426fb5558d0db7064f125094f71763 (patch) | |
| tree | b62f0045de4a03a280017e1fb67dba02691adf83 /llvm/lib | |
| parent | 64f772eac90d85351d9913abf6d2d7ac998c2431 (diff) | |
| download | bcm5719-llvm-1eeb2da7d4426fb5558d0db7064f125094f71763.tar.gz bcm5719-llvm-1eeb2da7d4426fb5558d0db7064f125094f71763.zip | |
Refactor: Simplify boolean conditional return statements in lib/Transforms/Vectorize (NFC).
Summary: Use clang-tidy to simplify boolean conditional return statements
Differential Revision: http://reviews.llvm.org/D10003
Patch by Richard<legalize@xmission.com>
llvm-svn: 251206
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | 5 | ||||
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 11 |
2 files changed, 5 insertions, 11 deletions
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp index 2381b63b767..c61a496b747 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -5215,9 +5215,8 @@ static bool isLikelyComplexAddressComputation(Value *Ptr, } static bool isStrideMul(Instruction *I, LoopVectorizationLegality *Legal) { - if (Legal->hasStride(I->getOperand(0)) || Legal->hasStride(I->getOperand(1))) - return true; - return false; + return Legal->hasStride(I->getOperand(0)) || + Legal->hasStride(I->getOperand(1)); } unsigned diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index 57b09caa5a4..5cd187877dd 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -168,10 +168,8 @@ static unsigned getAltOpcode(unsigned Op) { /// of an alternate sequence which can later be merged as /// a ShuffleVector instruction. static bool canCombineAsAltInst(unsigned Op) { - if (Op == Instruction::FAdd || Op == Instruction::FSub || - Op == Instruction::Sub || Op == Instruction::Add) - return true; - return false; + return Op == Instruction::FAdd || Op == Instruction::FSub || + Op == Instruction::Sub || Op == Instruction::Add; } /// \returns ShuffleVector instruction if instructions in \p VL have @@ -1990,10 +1988,7 @@ static bool shouldReorderOperands(int i, Instruction &I, return false; } // One opcode, put the instruction on the right. - if (ILeft) { - return true; - } - return false; + return ILeft != nullptr; } void BoUpSLP::reorderInputsAccordingToOpcode(ArrayRef<Value *> VL, |

