From: Chris Lattner <sabre@nondot.org>
Date: Sun, 28 Apr 2002 21:57:33 +0000 (+0000)
Subject: FIXME removed: malloc/alloca ALWAYS have a size argument
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=cc82cc174c4afa974450ac1e32ec85cb4d5324a8;p=oota-llvm.git

FIXME removed: malloc/alloca ALWAYS have a size argument


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2389 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/ExecutionEngine/Interpreter/Execution.cpp b/lib/ExecutionEngine/Interpreter/Execution.cpp
index 321003c1fdc..2a8c10506b5 100644
--- a/lib/ExecutionEngine/Interpreter/Execution.cpp
+++ b/lib/ExecutionEngine/Interpreter/Execution.cpp
@@ -731,14 +731,9 @@ void Interpreter::executeBrInst(BranchInst *I, ExecutionContext &SF) {
 
 void Interpreter::executeAllocInst(AllocationInst *I, ExecutionContext &SF) {
   const Type *Ty = I->getType()->getElementType();  // Type to be allocated
-  unsigned NumElements = 1;
 
-  // FIXME: Malloc/Alloca should always have an argument!
-  if (I->getNumOperands()) {   // Allocating a unsized array type?
-    // Get the number of elements being allocated by the array...
-    GenericValue NumEl = getOperandValue(I->getOperand(0), SF);
-    NumElements = NumEl.UIntVal;
-  }
+  // Get the number of elements being allocated by the array...
+  unsigned NumElements = getOperandValue(I->getOperand(0), SF).UIntVal;
 
   // Allocate enough memory to hold the type...
   // FIXME: Don't use CALLOC, use a tainted malloc.