From b84c5ae3d4d34d2e4cafbd7c1281de771b813a09 Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Thu, 10 Sep 2009 11:31:39 +0000 Subject: [PATCH] Add some braces to make newer GCCs happy and update CMakeLists. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@81443 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/CMakeLists.txt | 1 + lib/Analysis/MallocHelper.cpp | 2 +- lib/VMCore/Instructions.cpp | 8 +++++--- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/lib/Analysis/CMakeLists.txt b/lib/Analysis/CMakeLists.txt index 735e8a72ae7..d2333224f8f 100644 --- a/lib/Analysis/CMakeLists.txt +++ b/lib/Analysis/CMakeLists.txt @@ -22,6 +22,7 @@ add_llvm_library(LLVMAnalysis LoopInfo.cpp LoopPass.cpp LoopVR.cpp + MallocHelper.cpp MemoryDependenceAnalysis.cpp PointerTracking.cpp PostDominators.cpp diff --git a/lib/Analysis/MallocHelper.cpp b/lib/Analysis/MallocHelper.cpp index 5001110c6af..567268c6797 100644 --- a/lib/Analysis/MallocHelper.cpp +++ b/lib/Analysis/MallocHelper.cpp @@ -188,7 +188,7 @@ Value* llvm::getMallocArraySize(CallInst* CI) { Constant* CO = dyn_cast(MallocArg); BinaryOperator* BO = dyn_cast(MallocArg); - assert(isConstantOne(ElementSize) || CO || BO && + assert((isConstantOne(ElementSize) || CO || BO) && "getMallocArraySize and malformed malloc IR"); if (isConstantOne(ElementSize)) diff --git a/lib/VMCore/Instructions.cpp b/lib/VMCore/Instructions.cpp index a03587107cc..e52bc1c0518 100644 --- a/lib/VMCore/Instructions.cpp +++ b/lib/VMCore/Instructions.cpp @@ -462,7 +462,7 @@ static Value *checkArraySize(Value *Amt, const Type *IntPtrTy) { static Value *createMalloc(Instruction *InsertBefore, BasicBlock *InsertAtEnd, const Type *AllocTy, const Type *IntPtrTy, Value *ArraySize, const Twine &NameStr) { - assert((!InsertBefore && InsertAtEnd || InsertBefore && !InsertAtEnd) && + assert(((!InsertBefore && InsertAtEnd) || (InsertBefore && !InsertAtEnd)) && "createMalloc needs only InsertBefore or InsertAtEnd"); const PointerType *AllocPtrType = dyn_cast(AllocTy); assert(AllocPtrType && "CreateMalloc passed a non-pointer allocation type"); @@ -473,7 +473,7 @@ static Value *createMalloc(Instruction *InsertBefore, BasicBlock *InsertAtEnd, Value *AllocSize = ConstantExpr::getSizeOf(AllocPtrType->getElementType()); AllocSize = ConstantExpr::getTruncOrBitCast(cast(AllocSize), IntPtrTy); - if (!IsConstantOne(ArraySize)) + if (!IsConstantOne(ArraySize)) { if (IsConstantOne(AllocSize)) { AllocSize = ArraySize; // Operand * 1 = Operand } else if (Constant *CO = dyn_cast(ArraySize)) { @@ -483,13 +483,14 @@ static Value *createMalloc(Instruction *InsertBefore, BasicBlock *InsertAtEnd, AllocSize = ConstantExpr::getMul(Scale, cast(AllocSize)); } else { Value *Scale = ArraySize; - if (Scale->getType() != IntPtrTy) + if (Scale->getType() != IntPtrTy) { if (InsertBefore) Scale = CastInst::CreateIntegerCast(Scale, IntPtrTy, false /*ZExt*/, "", InsertBefore); else Scale = CastInst::CreateIntegerCast(Scale, IntPtrTy, false /*ZExt*/, "", InsertAtEnd); + } // Multiply type size by the array size... if (InsertBefore) AllocSize = BinaryOperator::CreateMul(Scale, AllocSize, @@ -498,6 +499,7 @@ static Value *createMalloc(Instruction *InsertBefore, BasicBlock *InsertAtEnd, AllocSize = BinaryOperator::CreateMul(Scale, AllocSize, "", InsertAtEnd); } + } // Create the call to Malloc. BasicBlock* BB = InsertBefore ? InsertBefore->getParent() : InsertAtEnd;