From 684b22df79c51114a12289e10a4063d5f02259a9 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 2 Aug 2007 16:53:43 +0000 Subject: wrap some long lines. Major offenders that are left include gvn, gvnpre, dse, and predsimplify. To see these, use: make check-line-length git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@40738 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/CodeGenPrepare.cpp | 4 ++-- lib/Transforms/Scalar/InstructionCombining.cpp | 3 ++- lib/Transforms/Scalar/LoopStrengthReduce.cpp | 7 ++++--- lib/Transforms/Scalar/LoopUnroll.cpp | 2 +- lib/Transforms/Scalar/LoopUnswitch.cpp | 6 ++++-- lib/Transforms/Scalar/RedundantLoadElimination.cpp | 3 ++- lib/Transforms/Scalar/ScalarReplAggregates.cpp | 2 +- 7 files changed, 16 insertions(+), 11 deletions(-) diff --git a/lib/Transforms/Scalar/CodeGenPrepare.cpp b/lib/Transforms/Scalar/CodeGenPrepare.cpp index ddc0007c00..c2a0787236 100644 --- a/lib/Transforms/Scalar/CodeGenPrepare.cpp +++ b/lib/Transforms/Scalar/CodeGenPrepare.cpp @@ -396,8 +396,8 @@ static bool OptimizeNoopCopyExpression(CastInst *CI, const TargetLowering &TLI){ /// OptimizeCmpExpression - sink the given CmpInst into user blocks to reduce /// the number of virtual registers that must be created and coalesced. This is -/// a clear win except on targets with multiple condition code registers (powerPC), -/// where it might lose; some adjustment may be wanted there. +/// a clear win except on targets with multiple condition code registers +/// (PowerPC), where it might lose; some adjustment may be wanted there. /// /// Return true if any changes are made. static bool OptimizeCmpExpression(CmpInst *CI){ diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index bb1b31e3ce..84ae26427b 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -7988,7 +7988,8 @@ bool InstCombiner::transformConstExprCastCall(CallSite CS) { &Args[0], Args.size(), Caller->getName(), Caller); cast(NC)->setCallingConv(II->getCallingConv()); } else { - NC = new CallInst(Callee, Args.begin(), Args.end(), Caller->getName(), Caller); + NC = new CallInst(Callee, Args.begin(), Args.end(), + Caller->getName(), Caller); if (cast(Caller)->isTailCall()) cast(NC)->setTailCall(); cast(NC)->setCallingConv(cast(Caller)->getCallingConv()); diff --git a/lib/Transforms/Scalar/LoopStrengthReduce.cpp b/lib/Transforms/Scalar/LoopStrengthReduce.cpp index 5bea783ff6..fea3c420d9 100644 --- a/lib/Transforms/Scalar/LoopStrengthReduce.cpp +++ b/lib/Transforms/Scalar/LoopStrengthReduce.cpp @@ -651,9 +651,10 @@ void BasedUser::RewriteInstructionToUseNewBase(const SCEVHandle &NewBase, Instruction *InsertPt = PN->getIncomingBlock(i)->getTerminator(); Code = InsertCodeForBaseAtPosition(NewBase, Rewriter, InsertPt, L); - // Adjust the type back to match the PHI. Note that we can't use InsertPt - // here because the SCEVExpander may have inserted its instructions after - // that point, in its efforts to avoid inserting redundant expressions. + // Adjust the type back to match the PHI. Note that we can't use + // InsertPt here because the SCEVExpander may have inserted its + // instructions after that point, in its efforts to avoid inserting + // redundant expressions. if (isa(PN->getType())) { Code = SCEVExpander::InsertCastOfTo(Instruction::IntToPtr, Code, diff --git a/lib/Transforms/Scalar/LoopUnroll.cpp b/lib/Transforms/Scalar/LoopUnroll.cpp index babfc2437e..25a4c4a79b 100644 --- a/lib/Transforms/Scalar/LoopUnroll.cpp +++ b/lib/Transforms/Scalar/LoopUnroll.cpp @@ -41,7 +41,7 @@ using namespace llvm; STATISTIC(NumCompletelyUnrolled, "Number of loops completely unrolled"); -STATISTIC(NumUnrolled, "Number of loops unrolled (completely or otherwise)"); +STATISTIC(NumUnrolled, "Number of loops unrolled (completely or otherwise)"); namespace { cl::opt diff --git a/lib/Transforms/Scalar/LoopUnswitch.cpp b/lib/Transforms/Scalar/LoopUnswitch.cpp index 6b4d6376f2..130db9fe41 100644 --- a/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/lib/Transforms/Scalar/LoopUnswitch.cpp @@ -425,7 +425,8 @@ static inline void RemapInstruction(Instruction *I, } } -// CloneDomInfo - NewBB is cloned from Orig basic block. Now clone Dominator Info. +// CloneDomInfo - NewBB is cloned from Orig basic block. Now clone Dominator +// Info. // // If Orig block's immediate dominator is mapped in VM then use corresponding // immediate dominator from the map. Otherwise Orig block's dominator is also @@ -1113,7 +1114,8 @@ void LoopUnswitch::SimplifyCode(std::vector &Worklist, Loop *L) { switch (I->getOpcode()) { case Instruction::Select: if (ConstantInt *CB = dyn_cast(I->getOperand(0))) { - ReplaceUsesOfWith(I, I->getOperand(!CB->getZExtValue()+1), Worklist, L, LPM); + ReplaceUsesOfWith(I, I->getOperand(!CB->getZExtValue()+1), Worklist, L, + LPM); continue; } break; diff --git a/lib/Transforms/Scalar/RedundantLoadElimination.cpp b/lib/Transforms/Scalar/RedundantLoadElimination.cpp index 8d63a39bd0..3b719df4f6 100644 --- a/lib/Transforms/Scalar/RedundantLoadElimination.cpp +++ b/lib/Transforms/Scalar/RedundantLoadElimination.cpp @@ -66,7 +66,8 @@ bool RLE::runOnBasicBlock(BasicBlock &BB) { bool MadeChange = false; // Do a top-down walk on the BB - for (BasicBlock::iterator BBI = BB.begin(), BBE = BB.end(); BBI != BBE; ++BBI) { + for (BasicBlock::iterator BBI = BB.begin(), BBE = BB.end(); + BBI != BBE; ++BBI) { // If we find a store or a free... if (LoadInst* L = dyn_cast(BBI)) { // We can't delete volatile loads diff --git a/lib/Transforms/Scalar/ScalarReplAggregates.cpp b/lib/Transforms/Scalar/ScalarReplAggregates.cpp index 52edcb67af..98fc399c71 100644 --- a/lib/Transforms/Scalar/ScalarReplAggregates.cpp +++ b/lib/Transforms/Scalar/ScalarReplAggregates.cpp @@ -50,7 +50,7 @@ namespace { static char ID; // Pass identification, replacement for typeid explicit SROA(signed T = -1) : FunctionPass((intptr_t)&ID) { if (T == -1) - SRThreshold = 128; + SRThreshold = 512; else SRThreshold = T; } -- cgit v1.2.3