[llvm-commits] [llvm] r59416 - /llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp

[Torok Edwin]

Author: edwin
Date: Sun Nov 16 11:21:25 2008
New Revision: 59416

URL: http://llvm.org/viewvc/llvm-project?rev=59416&view=rev
Log:
If SI->size() is 0, we are not allowed to dereference ->begin().
This fixed PR3078.


Modified:
    llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp

Modified: llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp?rev=59416&r1=59415&r2=59416&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp (original)
+++ llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp Sun Nov 16 11:21:25 2008
@@ -565,9 +565,11 @@
       // Add a parameter to the function for each element passed in.
       for (ScalarizeTable::iterator SI = ArgIndices.begin(),
              E = ArgIndices.end(); SI != E; ++SI) {
+        unsigned num = SI->size();
+        // not allowed to dereference ->begin() if size() is 0
         Params.push_back(GetElementPtrInst::getIndexedType(I->getType(),
-                                                           &*SI->begin(),
-                                                           SI->size()));
+                                                           num ? &*SI->begin(): 0,
+                                                           num));
         assert(Params.back());
       }