<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 29, 2016 at 5:05 PM, David Li 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">davidxl added a comment.<br>
<br>
please do not remove the no compression path. I think we should just keep your original change of making the iteration to a smaller number.  With the latest change plus 1024 iteration, the test time is ~20ms. I am ok with shorten the iteration further if we think 20ms is too costly.<br></blockquote><div><br></div><div>I can understand the value in testing the compression codepath (though it seems useful to actually test that it does anything - just testing that it roundtrips doesn't seem to demonstrate that the flag is actually causing the code to do anything different from when compression is disabled), but I still don't understand the point of the 1024 iterations.<br><br>My point of concern is not about whether we can afford 20ms, it's about whether there's something we're getting out of that. What extra coverage is the test providing by dealing with 1024*3 things rather than 3?<br><br>- David</div></div></div></div>