<div dir="ltr">LGTM as well. Let me know if you need someone to commit it.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Feb 8, 2018 at 10:30 AM Craig Topper 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"><div dir="ltr">Patch looks good to me.</div><div class="gmail_extra"></div><div class="gmail_extra"><br clear="all"><div><div class="m_-6077698520869245889gmail_signature" data-smartmail="gmail_signature">~Craig</div></div></div><div class="gmail_extra">
<br><div class="gmail_quote">On Thu, Feb 8, 2018 at 3:58 AM, David Woodhouse <span dir="ltr"><<a href="mailto:dwmw2@infradead.org" target="_blank">dwmw2@infradead.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Thu, 2018-02-08 at 11:27 +0000, Chandler Carruth wrote:<br>
> This looks pretty good to me, but Craig should probably have a look<br>
> before it lands.<br>
><br>
> If you don't have commit access, you may need to send it as an<br>
> attachment -- my email client at least destroys the patch file when<br>
> in-line so that I can't apply it and commit it easily for you.<br>
<br>
</span>I have had commit access in the past. I'll see if git-svn or I can<br>
remember my password ;)<br>
<br>
The other 32-bit build failure looks slightly more complex. It looks<br>
like there's a switch(sizeof(arg)) which is obviously (to me but not<br>
LLVM) constant, and the inline asm for the case which can never<br>
actually happen, would have bogus constraints because of the size.<br>
GCC optimises it away before ever caring, and LLVM doesn't.<br>
<br>
That was very late last night though; I'll take another look today.<br>
<br>
I'll also look at the per-function attribute to disable retpoline.<br>
</blockquote></div><br></div>
_______________________________________________<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>