
<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi David,<div class=""><br class=""></div><div class="">This is the first time we've really gone through the full proposal process, so this is a bit new for everyone. The decision making process was documented in the first proposal <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0001-LLVMDecisionMaking.md" class="">here</a>, and the relevant bits are steps 8 & 9:</div><div class=""><br class=""></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;" class=""><div class=""><div class="">8. When the discussion concludes, Chris and the review managers have a video chat to review the outcome of the discussion. The goal of this private discussion is to achieve consensus on an outcome between the review managers and Chris, but if that isn't possible, then Chris will tie break. The outcome may be Approve, Deny, Approve with Changes, or to kick it back to the pitch phase for more discussion.</div></div><div class=""><div class=""><br class=""></div></div><div class=""><div class="">9. A review manager writes up a summary of the outcome and shares that with the community on the llvm-dev. The outcome is added to the proposal in github to build a history of proposals and their outcomes.</div></div><div class=""><br class=""></div></blockquote>With that completed, the intent is that the decision of the review managers is the final decision.<div class=""><br class=""></div><div class="">I was probably overly brief in my on-list summary of the review managers decision. It is all captured now in the <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0002-BazelBuildConfiguration.md#review-meeting-notes" class="">proposal</a>, but the <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0002-BazelBuildConfiguration.md#decision-summary" class="">summary</a> is:</div><div class=""><br class=""></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;" class=""><div class=""><div class="">The review managers met to discuss the technical and community aspects of the proposal. It was the consensus of the review managers in attendance that the proposal, if approved, would impose little to no burden on the community, and provided material benefit to contributors and downstream users.</div><div class=""><br class=""></div><div class="">The committee also agreed that the Bazel build files should be included under the top-level <a href="http://github.com/llvm/llvm-project/utils" class="">github.com/llvm/llvm-project/utils</a> directory as they relate to multiple subprojects in LLVM.</div></div><div class=""><br class=""></div></blockquote>In the <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0002-BazelBuildConfiguration.md#discussion-notes" class="">discussion notes</a> we did discuss source location as it relates to gn as well:<div class=""><br class=""></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;" class=""><div class="">In the interest of this proposal resulting in the correct decision, we agreed the Bazel files should live under the utils directory at the root of the llvm-project repository, and we think GN should move there as well.</div></blockquote><div class=""><br class=""></div>Hope this answers all your questions.<div class=""><br class=""></div><div class="">-Chris<br class=""><div class=""><div class=""><div class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Mar 15, 2021, at 12:29 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Thanks for the update Chris - could you summarize what this means for the proposal/what stage in the proposal process this is? Does this represent approval, and the patch should now be submitted without further high level design review (that is covered by the proposal review)? Or are there further steps?<br class=""><br class="">(does the approval indicate where these files should live? Next to the gn files? A new top level location? or is that still up to further community review)</div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 9:22 AM Chris Bieneman via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;" class="">Hello LLVM-Dev,<div class=""><br class=""></div><div class="">Last week the review managers met to discuss this proposal. I've updated the proposal document with a summary of the meeting. You can find the proposal online <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0002-BazelBuildConfiguration.md" target="_blank" class="">here</a>.</div><div class=""><br class=""></div><div class="">The TL;DR is that the review managers agreed the proposal should be approved.</div><div class=""><br class=""></div><div class="">Thank you everyone who participated in the conversations around this proposal, and especially Geoffrey for putting the proposal together and shepherding it along.</div><div class=""><br class=""></div><div class="">-Chris<br class=""><div class=""><br class=""><blockquote type="cite" class=""><div class="">On Feb 19, 2021, at 1:46 PM, Geoffrey Martin-Noble <<a href="mailto:gcmn@google.com" target="_blank" class="">gcmn@google.com</a>> wrote:</div><br class=""><div class=""><div dir="ltr" class="">A reminder that the review period for this ends 2021-02-23, this coming Tuesday. Rest assured that if you expressed opinions in the previous RFC threads then review managers will also consider those points when discussing. We're not going to skip some point just because it wasn't posted in the correct thread :-D<div class=""><br class=""></div><div class="">Best,</div><div class="">Geoffrey</div></div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 10, 2021 at 3:44 AM Renato Golin <<a href="mailto:rengolin@gmail.com" target="_blank" class="">rengolin@gmail.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class=""><div dir="ltr" class="">On Tue, 9 Feb 2021 at 21:00, Geoffrey Martin-Noble <<a href="mailto:gcmn@google.com" target="_blank" class="">gcmn@google.com</a>> wrote:</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"><div dir="ltr" class=""><div class="gmail_quote"><div class="">To expand a bit on Eric's response, the intent here is *not* to make Bazel a supported build system for LLVM or to replace CMake (which I believe the proposal makes clear), but rather to enable Bazel usage and shared configuration for people and projects that already use it. I do not expect that Bazel will cover all the use cases currently supported by LLVM CMake any time soon (ever?).I don't work on Bazel itself, so have no insight on the support plan for those architectures. Only developers interested in working with Bazel would be expected to use or update the configuration, so lack of support for specific architectures should not affect things, I think.</div></div></div></blockquote><div class=""><br class=""></div><div class="">My views exactly. Bazel will not be a "supported" build system and doesn't need to build on all platforms and environments LLVM builds. It should only concern people that actually use Bazel and be completely transparent to the rest who don't.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class=""><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">

</blockquote></div></div>

</blockquote></div></div>

</blockquote></div>

</div></blockquote></div><br class=""></div></div>_______________________________________________<br class="">

LLVM Developers mailing list<br class="">

<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" class="">llvm-dev@lists.llvm.org</a><br class="">

<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br class="">

</blockquote></div>

</div></blockquote></div><br class=""></div></div></div></div></body></html>