I updated the tarball with the patch.  I also did cleaning of the other routines, to be sure the routines return proper values.<br><br>Sherry<br> <br><br><div class="gmail_quote">On Thu, Dec 30, 2010 at 6:01 PM, Jed Brown <span dir="ltr"><<a href="mailto:jed@59a2.org">jed@59a2.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im"><div class="gmail_quote">On Thu, Dec 23, 2010 at 09:18, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>petsc configure will now download superlu/<span>superlu_dist</span> directly from<br>
this location as we won't have any more local patches..</div></blockquote></div><br></div><div>We need one more patch (attached) to compile with Clang.  This only fixes the fatal errors, none of the calls to these functions check the return value so maybe they should just return void.</div>

<div><br></div><div>Additionally, there are lots of functions returning non-void but lacking a return statement.  This is not valid C, but Clang (and other compilers, evidently) happen to only issue warnings for these violations.</div>

</blockquote></div><br>