<div dir="ltr">Patch looks good to me.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div>
<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 class="">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>