commit 7b15ea3405a02cec9951c8e491abb4688dee57c1
author Shih-wei Liao <sliao@google.com> Wed Sep 12 11:01:26 2012 -0700
committer Shih-wei Liao <sliao@google.com> Fri Sep 21 18:15:39 2012 -0700
tree 135a1983a984b36c019f1e1b3af28c73395320a8
parent 60ca7e1fbf7d8310387de5091ec76af538ecd480

Fix a bug in ABCExpandVAArgPass.

eraseFromParent() invalidates the current iterator.

Change-Id: I969b5bad66410fc4c91d7a39f2674f8b84942185

lib/AndroidBitcode/ABCExpandVAArgPass.cpp

diff --git a/lib/AndroidBitcode/ABCExpandVAArgPass.cpp b/lib/AndroidBitcode/ABCExpandVAArgPass.cpp
index 7b89ced..1f0debf 100644
--- a/lib/AndroidBitcode/ABCExpandVAArgPass.cpp
+++ b/lib/AndroidBitcode/ABCExpandVAArgPass.cpp
@@ -19,6 +19,7 @@
 
 #include "bcc/AndroidBitcode/ABCExpandVAArgPass.h"
 
+#include <llvm/ADT/STLExtras.h>
 #include <llvm/Instructions.h>
 #include <llvm/Support/InstIterator.h>
 
@@ -33,7 +34,8 @@
 
   // process va_arg inst
   for (llvm::inst_iterator inst = llvm::inst_begin(pFunc),
-          inst_end = llvm::inst_end(pFunc); inst != inst_end; inst++) {
+          inst_end = llvm::inst_end(pFunc), next_inst = llvm::next(inst);
+       inst != inst_end; inst = next_inst++) {
     if (inst->getOpcode() == llvm::Instruction::VAArg) {
       llvm::Value *v = expandVAArg(&*inst);
       inst->replaceAllUsesWith(v);