<div dir="ltr">(Disclaimer: I haven't looked at the branch in question, so please ignore if redundant)  It would be good to commit the test case too, if you haven't already added it.</div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Tue, Oct 15, 2013 at 4:00 PM, Rob Latham <span dir="ltr"><<a href="mailto:robl@mcs.anl.gov" target="_blank">robl@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Mon, Oct 14, 2013 at 09:54:56PM -0500, Rob Latham wrote:<br>
> good news is that I think everything "important" is happening<br>
> correctly.  It's just that when the status object is updated it's<br>
> picking up an overflowed integer.  status.count is MPI_Count big, so<br>
> that's not the source of the overflow.  will look once more at my<br>
> mountain of clang -Wshorten-64-to-32 related warnings in the morning.<br>
<br>
</div>Jeff found the casts to int and I've added his two patches to my<br>
branch in -review.   Jeff and I now pass Jeff's testcase.<br>
<div class="HOEnZb"><div class="h5"><br>
==rob<br>
<br>
--<br>
Rob Latham<br>
Mathematics and Computer Science Division<br>
Argonne National Lab, IL USA<br>
</div></div></blockquote></div><br></div>