<div class="gmail_extra"><div class="gmail_quote">On Sat, Aug 11, 2012 at 4:20 PM, Michael Liao <span dir="ltr"><<a href="mailto:michael.liao@intel.com" target="_blank" class="cremed">michael.liao@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sat, 2012-08-11 at 16:06 -0700, Chandler Carruth wrote:<br>
> On Sat, Aug 11, 2012 at 3:49 PM, Michael Liao <<a href="mailto:michael.liao@intel.com" class="cremed">michael.liao@intel.com</a>><br>
> wrote:<br>
>         Hi<br>
><br>
>         Sorry, PR13577 and recent failures of compiler-rt failures is<br>
>         introduced<br>
>         by r161687. The root cause is that FCMOV (only used for 80-bit<br>
>         double)<br>
>         only supports a subset of X86 conditions. The attached patch<br>
>         fixes that<br>
>         by checking whether it's a CMOV on f80 and whether the<br>
>         condition is<br>
>         valid. A minimal test case (copysign) for PR13577 is also<br>
>         added.<br>
><br>
><br>
> Hmm, my diagnosis of that PR may have been a bit off -- you should<br>
> check whether this patch actually fixes it, as it seems to potentially<br>
> predate r161687. Fixing it may however re-use this logic.<br>
><br>
<br>
</div>I could compile lib/divxc3.c with llvm/clang with this patch. I also<br>
checked out r161491 (no r161493 in llvm itself) and cannot reproduce<br>
that issue. I could confirm that issue (PR13577) is caused by r161687<br>
alone.</blockquote><div><br></div><div>Awesome. =] Please update the PR with this info and close it when you submit! Thanks for the fast fix!! =D </div></div></div>