<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>Overall, +1 from me.</p>
<p>I have a few concerns about the detail, but adopting this with
the attitude that we're expecting to iterate on the process itself
seems like a step in the right direction. <br>
</p>
<p>I'll simply list my concerns as I think each of these has been
expanded on down thread by others.</p>
<p>1) Length of review period - two weeks is too short.</p>
<p>2) Who selects review managers - I'd be fine with Chris having
final call. I also think concerns about selection can largely be
handled on a case by case basis as part of review feedback.</p>
<p>3) I'd prefer llvm-dev over another mailing list for now. We
could switch later, but I don't see a strong reason to switch now.</p>
<p>4) I think it needs to be very clear that reviewers are only
expected to have read the current proposal. It should be the
responsibility of author to summarize previous discussion. I
consider the "this was discussed in X old thread" an anti pattern
for this type of discussion.</p>
<p>Philip<br>
</p>
<p><br>
</p>
<div class="moz-cite-prefix">On 6/2/20 1:19 PM, Chris Lattner via
llvm-dev wrote:<br>
</div>
<blockquote type="cite"
cite="mid:B9F356DA-DB9B-4226-84AE-7238008C0339@nondot.org">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
Hi all,
<div class=""><br class="">
</div>
<div class="">Following up on the extensive discussions since
January, many of us would like to put in place a process to
improve LLVM’s decision making process for contentious issues.
I’ve put together a proposal for how this works, and am
recursively using it to get feedback on the process itself.
Thank you to the many people who contributed great ideas and
improvements during the pitch phases and early drafts of the
doc.</div>
<div class=""><br class="">
</div>
<div class="">Because this is a weird case, I’m not setting up the
standard review manager team for this. We’ll wing it, and if it
doesn’t work out, we can try again.</div>
<div class=""><br class="">
</div>
<div class="">-Chris</div>
<div class=""><br class="">
</div>
<div class="">——</div>
<div class=""><br class="">
</div>
<div class="">
<div class="">Hello LLVM community,</div>
<div class=""><br class="">
</div>
<div class="">The review of "Introduce a new LLVM process to
resolve contentious decisions" begins now and runs through </div>
<div class="">June 12, 2020. The proposal is <a
href="https://github.com/llvm/llvm-www/blob/master/proposals/LP0001-LLVMDecisionMaking.md"
class="" moz-do-not-send="true">available online</a>.</div>
<div class=""><br class="">
</div>
<div class="">Feedback is an important part of the LLVM Proposal
process. All review feedback</div>
<div class="">should be either on this forum thread or, if you
would like to keep your feedback</div>
<div class="">private, directly to one of the review managers.</div>
<div class=""><br class="">
</div>
<div class="">**What goes into a review?**</div>
<div class=""><br class="">
</div>
<div class="">The goal of the review process is to improve the
proposal under review through</div>
<div class="">constructive criticism and, eventually, determine
the direction of LLVM. When </div>
<div class="">writing your response, here are some questions you
might want to answer in your</div>
<div class="">review:</div>
<div class=""><br class="">
</div>
<div class="">* What is your evaluation of the proposal? What
positive or negative </div>
<div class=""> implications would accepting this have?</div>
<div class="">* Do you have experience from other communities
that relates to this </div>
<div class=""> issue and is important to consider?</div>
<div class="">* How involved have you been in the LLVM
project? Frequent contributor, </div>
<div class=""> occasional contributor, user of LLVM
libraries, user of LLVM-based tools,</div>
<div class=""> or other?</div>
<div class="">* Self Evaluation: How much effort did you put
into your review and how</div>
<div class=""> knowledgeable are you about this area? For
example, a quick reading or an </div>
<div class=""> in-depth study?</div>
<div class=""><br class="">
</div>
<div class="">In addition to your opinion and thoughts, please
include any additional </div>
<div class="">framing that may be useful.</div>
<div class=""><br class="">
</div>
<div class="">Thank you,</div>
<div class=""><br class="">
</div>
<div class="">Chris Lattner</div>
<div class="">Review Manager</div>
</div>
<div class=""><br class="">
</div>
<div class=""><br class="">
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
LLVM Developers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>
<a class="moz-txt-link-freetext" href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a>
</pre>
</blockquote>
</body>
</html>