<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, May 3, 2015 at 6:18 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Richard Smith wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
On Fri, May 1, 2015 at 6:24 PM, Richard Trieu <<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a><br></span><span class="">
<mailto:<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>>> wrote:<br>
<br>
    Tom,<br>
<br>
    These are patches that fix crashes that I have encountered while<br>
    working on Clang.  I believe that they would be good for 3.6.1.  I<br>
    included revision numbers, bug reports, and authors below.<br>
<br>
    These two patches fix crash on invalids involving template diffing.<br>
    r226983 - PR21758 (rtrieu)<br>
    r230603 - PR22017 (rtrieu)<br>
<br>
    Other small patches that fix crashes.  Patch authors CCed<br>
    r232425 - (rtrieu)<br>
    r232449 - PR22929 (dexonsmith)<br>
    r232579 - PR22940 (nicholas)<br>
    r232675 - PR22931 (rtrieu)<br>
    r232786 - PR22959 (majnemer)<br>
    r232794 - PR22960 (nicholas)<br>
<br>
<br>
Hi Tom,<br>
<br>
All of the above are approved for branch other than nicholas'  r232579<br>
</span></blockquote>
<br>
r232579 is a clang irgen change with no llvm component. I'd like someone familiar with clang to sign off.</blockquote><div><br></div><div>Gah, off-by-one error, I meant r232449, which is an LLVM-only change :) 232579 LGTM for branch.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
and r232794 -- those are LLVM changes, you'll need someone else to sign<br>
off on those.<br>
</blockquote>
<br></span>
r232794 is really quite safe, I'd sign off on it. Looking at code owners, this would appear to not match anything before Chris Lattner's fallback entry.<span class="HOEnZb"><font color="#888888"><br>
<br>
Nick<br>
</font></span></blockquote></div><br></div></div>