Looks good! Thanks for doing this.<br>
<br>+  // * If the optimization passes use inter procedural information to brake<br>+  //   a function, we want to contition with the original function. Otherwise<br><br>"inter procedural" -> "interprocedural"<br>

"brake" -> "break"<br>"contition" -> um, "continue" maybe?<br><br>+  std::vector<Function*> FuncsOnClone;<br><br>Do you want to use a SmallVector here?<br><br>+  for (unsigned i = 0, e = Funcs.size(); i !=e; ++i) {<br>

<br>Add space after "!=" before "e"<br><br>Nick<br><br><div class="gmail_quote">On 25 July 2010 07:13, Rafael Espindola <span dir="ltr"><<a href="mailto:espindola@google.com">espindola@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">> it looks like you forgot to attach it...<br>
<br>
</div>Arg.<br>
<br>
Attached now. I tested it on the testcase that caused it to crash. It<br>
is now able to correct split the functions into opt/non-opt and then<br>
crashes when handling the basic blocks.<br>
<br>
> Ciao,<br>
><br>
> Duncan.<br>
<div><div></div><div class="h5"><br>
Cheers,<br>
--<br>
Rafael Ávila de Espíndola<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br>