<div dir="ltr"><div><div>Hi Hans -<br><br></div>If you can do the merge to 3.8, I'd appreciate it.<br><br></div>Thanks!<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 20, 2016 at 10:55 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Jan 20, 2016 at 9:41 AM, Sanjay Patel via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: spatel<br>
> Date: Wed Jan 20 11:41:14 2016<br>
> New Revision: 258325<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=258325&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=258325&view=rev</a><br>
> Log:<br>
> [LibCallSimplifier] don't get fooled by a fake sqrt()<br>
><br>
> The test case will crash without this patch because the subsequent call to<br>
> hasUnsafeAlgebra() assumes that the call instruction is an FPMathOperator<br>
> (ie, returns an FP type).<br>
><br>
> This part of the function signature check was omitted for the sqrt() case,<br>
> but seems to be in place for all other transforms.<br>
><br>
> Before:<br>
> <a href="http://reviews.llvm.org/rL257400" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL257400</a><br>
> ...we would have needlessly continued execution in optimizeSqrt(), but the<br>
> bug was harmless because we'd eventually fail some other check and return<br>
> without damage.<br>
><br>
> This should fix:<br>
> <a href="https://llvm.org/bugs/show_bug.cgi?id=26211" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=26211</a><br>
><br>
> Differential Revision: <a href="http://reviews.llvm.org/D16198" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16198</a><br>
<br>
Please go ahead and merge this to 3.8 once it's been on the buildbots<br>
for a while, or let me know if you'd prefer me to do it.<br>
<br>
Thanks,<br>
Hans<br>
</blockquote></div><br></div>