<div class="gmail_quote">On 13 January 2010 16:20, 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">> Looks good to me. Thanks Rafael!<br>
<br>
</div>OK, it was not so simple because of the strict argument order. Fixed.<br>
I also fixed the very long lines is Makefile.programs.<br></blockquote><div><br>Yuck. Without proper flags processing, this whole script should probably just use environment variables instead. Anyways, thanks for the refactoring, this looks much better.<br>

<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Note that RunSafelyAndStable.sh cannot take options.<br></blockquote><div><br>No, but I'm not sure it needs to. It takes <program> <args...> which means that "prog foo" could become "wrapper prog foo" with no difference in behaviour. But after taking another look, I don't see any of the makefiles in llvm-test actually using GET_STABLE_NUMBERS. I guess you don't really need to support that. :)<br>

</div></div><br>So with that said, this patch <i>once again</i> looks fine, but then, so did all the others ;)<br><br>Nick<br>