<div dir="ltr">Sorry, I don't think I was clear enough.<div><br></div><div>When I make AVR the default target triple, most of the generic CodeGen tests pass, but some don't. The easiest solution was to make X86 the default target so the generic CodeGen tests run under it, and that way the AVR specific tests still run.</div><div><br></div><div>The bug causing the tests to fail is not because the default target hasn't been built, it's an assertion error hit inside the AVR ISel code.</div><div><br></div><div>Once the generic suite passes, I will make AVR the default target on the builders.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 4, 2017 at 12:17 PM, Mehdi Amini 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 style="word-wrap:break-word"><span class=""><br><div><blockquote type="cite"><div>On Feb 3, 2017, at 5:15 PM, Mehdi Amini via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="m_2525265309773081030Apple-interchange-newline"><div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><blockquote type="cite"><div><br class="m_2525265309773081030Apple-interchange-newline">On Feb 3, 2017, at 5:14 PM, Matthias Braun <<a href="mailto:mbraun@apple.com" target="_blank">mbraun@apple.com</a>> wrote:</div><br class="m_2525265309773081030Apple-interchange-newline"><div><div style="word-wrap:break-word"><br><div><blockquote type="cite"><div>On Feb 3, 2017, at 12:45 PM, Dylan McKay via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="m_2525265309773081030Apple-interchange-newline"><div><div dir="ltr"><div class="m_2525265309773081030markdown-here-wrapper"><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">The builder isn’t marked as experimental so I think the expectation is that people keep it green and contact the bot owner if they need help figuring out why their change makes it red. That said, it sounds a bit odd to have a non-experimental builder for an experimental backend.</p></blockquote><p style="margin:0px 0px 1.2em!important">I see. I had followed the generic<span class="m_2525265309773081030Apple-converted-space"> </span><a href="http://llvm.org/docs/HowToAddABuilder.html" target="_blank">How to add a builder</a><span class="m_2525265309773081030Apple-converted-space"> </span>docs, which doesn’t mention the concept of an experimental buildbot. I’ll send a patch to mention it.</p><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">If you want to do the same, then you’ll need to add an InformativeMailNotifier to<span class="m_2525265309773081030Apple-converted-space"> </span><a href="http://llvm.org/svn/llvm-project/zorg/trunk/buildbot/osuosl/master/config/status.py" target="_blank">http://llvm.org/svn/llvm-<wbr>project/zorg/trunk/buildbot/<wbr>osuosl/master/config/status.py</a><wbr>.</p></blockquote><p style="margin:0px 0px 1.2em!important">Nice! Exactly what I was looking for.</p><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">If we would believe the AVR backend is stable enough, such that users can rely on it and such that other developers are unlikely to trigger bugs in the AVR backend, the AVR backend should most likely be promoted to a stable backend.</p></blockquote><p style="margin:0px 0px 1.2em!important">In general, I’ve found that almost all of the time that the AVR build breaks, it’s been something pretty small which also caused a bunch of other targets to fail also, which I suppose is a good sign. On the topic, I plan on following up on promoting the backend to stable once the current effort of enabling AVR in Rust is complete and we’ve ironed out any bugs found in usage.</p><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">As a result of this, I would also expect buildbots of the AVR backend to not send any emails to the general public, but to instead send emails to the buildbot owner and maintainer of the AVR backend.</p></blockquote><p style="margin:0px 0px 1.2em!important">Agree with this</p><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">+1. The silent staging buildbot is what you want I believe<br><a href="http://lists.llvm.org/pipermail/lldb-commits/Week-of-Mon-20151012/024214.html" target="_blank">http://lists.llvm.org/<wbr>pipermail/lldb-commits/Week-<wbr>of-Mon-20151012/024214.html</a></p></blockquote><p style="margin:0px 0px 1.2em!important">That sounds good. My plan is to make the buildbot a staging bot, and then be the sole receiver of emails from it.</p><blockquote style="margin:1.2em 0px;border-left-width:4px;border-left-style:solid;border-left-color:rgb(221,221,221);padding:0px 1em;color:rgb(119,119,119);quotes:none"><p style="margin:0px 0px 1.2em!important">If I was in this position, I’d also configure the bot to build<span class="m_2525265309773081030Apple-converted-space"> </span><em>only</em><span class="m_2525265309773081030Apple-converted-space"> </span>the AVR backend. That’s help make sure that an email does get send when a test fails in the X86 backend.</p></blockquote><p style="margin:0px 0px 1.2em!important">I would love to do this, but there’s a bug in the backend which causes a few of the Generic CodeGen tests to fail. To work around this, I leave X86 as the default target for now. I’m definitely planning on updating this once I’ve fixed the bug.</p><div><br></div></div></div></div></blockquote><div><br></div><div>This usually happens when LLVM_DEFAULT_TARGET_TRIPLE is not explicitely set and you end up with your host machine as default while not building the x86 target. If you set LLVM_DEFAULT_TARGET_TRIPLE to some AVR ones the failure should go away (otherwise complain and file bugs).</div></div></div></div></blockquote><br></div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">Actually I believe you have to set it to empty to disable “generic” tests.</div></div></blockquote><br></div></span><div>Confirmed:</div><div><br></div><div>$ cat llvm/test/CodeGen/Generic/lit.<wbr>local.cfg <br>if not config.target_triple:<br>    config.unsupported = True<br><br></div><div><br></div><div>— </div><div>Mehdi</div><div><br></div><br></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>