<div dir="ltr">We could make lit tests more portable by unconditionally setting execute_external to False in lit.common.cfg. This would use the lit shell on all platforms, and make it impossible to accidentally write tests that don't run on Windows from Posix.<div><br></div><div>We have a long term goal to do this anyway: <a href="https://bugs.llvm.org//show_bug.cgi?id=5241">https://bugs.llvm.org//show_bug.cgi?id=5241</a> Want to be the first LLVM project to do it?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 30, 2017 at 8:38 AM, Kostya Serebryany via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@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"><div dir="ltr">Rafael, Filipe, <div><br></div><div>I am looking at the fixes you apply to sanitizer tests and they worry me. </div><div>(e.g. <a href="https://reviews.llvm.org/D31498" rel="noreferrer" style="font-size:12.8px" target="_blank">https://reviews.llvm.<wbr>org/D31498</a>)</div><div>The fixes are mostly mechanical and thus every single change looks safe,</div><div>but given the amount of changes there is large risk to cripple some of the tests</div><div>in a way that they will stop detecting failures. </div><div><br></div><div>When I write a test for new functionality, I always verify that the test fails w/o the implementation</div><div>of that functionality (i.e. that the test catches the failure). I hope most of the tests are developed this way too. But when you modify the tests, I bet you only verify that the passing test is still passing. Am I right? If yes, this is not the way to modify tests, unfortunately. </div><div><br></div><div>So, my question is: did you consider modifying the environment to closer match what already works well on posix, instead of massively changing tests? </div><div><br></div><div>Thanks, </div><div><br></div><div>--kcc </div></div>
<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>