<div dir="ltr">> <span style="font-size:12.8px">This test will fail as soon as anyone builds the AVR back-end without</span><br style="font-size:12.8px"><span style="font-size:12.8px">access to AVR hardware.</span><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I added a config file that ignored the tests if AVR was not built? On top of that, the tests were only supposed to run if 'AVRLIT_BOARD' and 'AVRLIT_PORT' is set. Clearly it didn't work properly (due to my inexperience with Python)</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">If I were to fix that, it wouldn't stop the tests from breaking if AVR was made non-experimental?</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">> </span><span style="font-size:12.8px">do not add *any* execution tests to the LLVM tree</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Do we have anything formal thing about this? I'd like to have a conversation about it. I agree it isn't something we really do currently, but I believe there is real value in adding these tests in tree.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">> </span><span style="font-size:12.8px">Reverted in r289651</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks for that!</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 15, 2016 at 2:35 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 14 December 2016 at 13:32, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br>
> Hi Dylan,<br>
><br>
> This test will fail as soon as anyone builds the AVR back-end without<br>
> access to AVR hardware.<br>
><br>
> It's also guaranteed to fail whenever the AVR back-end moves out of<br>
> experimental (when everyone will build it by default).<br>
><br>
> Please, revert this patch and do not add *any* execution tests to the LLVM tree.<br>
<br>
</span>Oh well, it seems I was being optimistic. It breaks now because no one<br>
else has the AVRLIT_PORT.<br>
<br>
<a href="http://lab.llvm.org:8011/builders/clang-cmake-aarch64-42vma/builds/2086/steps/ninja%20check%201/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/clang-cmake-aarch64-<wbr>42vma/builds/2086/steps/ninja%<wbr>20check%201/logs/stdio</a><br>
<br>
Reverted in r289651.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br></div>