<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, May 17, 2016 at 5:08 PM Mehdi Amini via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Seriously I absolutely don't get the "start a new revision" thing because the initial email was not on llvm-commit. Really what is the rational?<br>
<br>
We are losing the history and the threading for the review, this is really annoying.<br>
<br>
So big -1<br></blockquote><div><br></div><div>I don't care very much about starting a new revision.</div><div><br></div><div>Justin and others who don't use phabricator asked for this because that is what causes an email with a patch file to be sent to llvm-commits, and so I've been trying to encourage it based on their request.</div><div><br></div><div>There is still the fact that even the initial phab revision has essentially no context. I've skimmed thin three or four times and I'm not sure yet what the motivation is... I'm sure it has one, I just can't find it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
--<br>
Mehdi<br>
<br>
<br>
> On May 17, 2016, at 4:49 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:<br>
><br>
> chandlerc added a subscriber: chandlerc.<br>
> chandlerc added a comment.<br>
><br>
> Folks, llvm-commits wasn't even on the original version of this patch, and there is essentially no high-level description of the problem being solved or the motivation of the patch.<br>
><br>
> Can you start a fresh revision in phab and actually include a *lot* more context? I suspect many, many people will be very interested in something as impactful as a new variant of 'unnamed_addr' and they may (like me) have completely missed the discussion taking place here or lack significant context for it.<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D20260" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20260</a><br>
><br>
><br>
><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>