<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">I think that’s what Kate meant when she said “Will do.” to Todd.<div class=""><br class=""></div><div class="">Sean</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Oct 16, 2014, at 5:53 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Fri, Oct 3, 2014 at 8:43 PM, Todd Fiala <span dir="ltr" class=""><<a href="mailto:todd.fiala@gmail.com" target="_blank" class="">todd.fiala@gmail.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="">Hey Kate,<div class=""><br class=""></div><div class="">Might you be able to put up the patches (say on <a href="http://reviews.llvm.org/" target="_blank" class="">reviews.llvm.org</a>) a bit ahead of time and we can maybe have a chance to address any issues on the Linux side before it goes into top of tree?</div></div></blockquote></div><br class="">Moreover, it is totally unacceptable to drop code without review like this.</div><div class="gmail_extra"><br class=""></div><div class="gmail_extra">Please post the patch for review, and review it in the open.</div><div class="gmail_extra"><br class=""></div><div class="gmail_extra">Look at how the ARM64 backend was posted. It wasn't just "merged back". =/</div></div>
_______________________________________________<br class="">lldb-dev mailing list<br class=""><a href="mailto:lldb-dev@cs.uiuc.edu" class="">lldb-dev@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev<br class=""></div></blockquote></div><br class=""></div></body></html>