<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 26, 2015 at 2:57 AM, Charlie Turner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chatur01 updated this revision to Diff 41223.<br>
chatur01 added a comment.<br>
<br>
Added a test-case. This might violate the test guidelines of only comitting robust test cases. By definition I can't do that for a non-determinism bug, but I'm running the test case 5 times, which at least locally was always enough to catch the non-determinism on my computer, of course I can't guarantee that.<br></blockquote><div><br></div><div>No need to have the test actually run 5 times - once is enough to demonstrate the intended behavior (now that there's an intention to the ordering of the output). If someone refactored/broke this later, this test doesn't have to catch it immediately (that person introduced uncertainty in the output - as you say, you can't guarantee a test for non-determinism, but you can say "if this test fails, the program isn't behaving as intended")<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm requesting a sanity check on my test case before comitting it.<br></blockquote><div><br></div><div>If that's the smallest example of unstable output, then it's fine by me - I don't know enough about the actual code here to know if that's the case so I'll take yours/James's word for it. But drop the extra runs - just run it once.<br><br>(do no other tests cover the ordering of this output already? What was the buildbot failure you saw previously that lead you to fix this?)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Many thanks again David for your review.<br>
<span class=""><br>
<br>
<a href="http://reviews.llvm.org/D14981" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14981</a><br>
<br>
Files:<br>
  include/llvm/Analysis/VectorUtils.h<br>
  lib/Analysis/VectorUtils.cpp<br>
  lib/Transforms/Vectorize/LoopVectorize.cpp<br>
</span>  test/Transforms/LoopVectorize/AArch64/deterministic-type-shrinkage.ll<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>