<div dir="ltr"><div dir="ltr">On Mon, 29 Nov 2021 at 12:36, Simon Moll <<a href="mailto:Simon.Moll@emea.nec.com">Simon.Moll@emea.nec.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> > 1. Connect the VE worker to main buildbot instance.<br>
> > 2. Add `VE` to the LLVM target list, thereby making it official<br>
> > (D113247).<br>> This will trigger all bots that don't restrict targets to build VE,<br>
> and you may get some failures here already.<br>
I see no harm in moving clang-ve-ninja to main buildbot while we are<br>
watching the ramifications play out for all builders. Builders don't<br>
interfere with one another. There will only be better coverage.<br></blockquote><div><br></div><div>That's not what I mean. I also see no harm in moving the VE bot before the move, as it's meant to be passing all the time.</div><div><br></div><div>What I meant is that the move to production (D113247) could break other bots that will build VE if they don't have an exclusive list of targets to build.</div><div><br></div><div>Not because of the VE backend, we know it's fine (due to VE's own buildbot). But because the other bots may have a build option/environment that you haven't tested and will need to be sorted quickly.</div><div><br></div><div>Most likely nothing will happen, but I'm just making sure we expect a potential problem, and perhaps give it a day's worth of waiting after moving to production to continue.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This only matters for the clang-ve-ninja builder. The llvm-zorg patch<br>
moves 'VE' from the experimental target to the targets_to_build cmake<br>
option for clang-ve-ninja and enables reporting on failure. This will<br>
not effect other bots.<br></blockquote><div><br></div><div>Ack.<br></div></div></div>