<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 22, 2016 at 12:11 PM Mandeep Singh Grang via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mgrang added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Note: There are no associated tests with this patch as it is very difficult (not possible?) to detect "reverse" iteration since the relative order of elements in the SmallPtrSet is always undefined.<br class="gmail_msg">
I would be happy to write tests for this if someone can suggest a good way of testing it. Otherwise the biggest test for this is to enable it in the buildbot and let the unit tests fail due to non-deterministic codegen.<br class="gmail_msg"></blockquote><div><br></div><div>Yeah, that's fine - I think of this like a valgrind buildbot. If we find errors with this mode, there should be a test that exercises/demonstrates the problem (in the common case there will, as you say, /already/ be a test - that fails on the reverse order and we fix it and move on, but if say someone finds a non-determinism in their custom compiler workload, we can now ask them for a test that fails under one of forward or reverse (and if it doesn't fail under either we can consider whether it is a very special nondeterminism, if we can add better coverage, etc))</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26718" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26718</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
llvm-commits mailing list<br class="gmail_msg">
<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="gmail_msg">
</blockquote></div></div>