<div dir="ltr"><div>While you're here, you might as well fix the calls to EmitUnaryFloatFnCall to get the function name from TLI as well.</div><div><br></div><div><div>-    if (ConstantFP *Op1C = dyn_cast<ConstantFP>(Op1)) {</div>
<div>+    ConstantFP *Op1C = dyn_cast<ConstantFP>(Op1);<br></div><div>+    if (Op1C) {</div></div><div><br></div><div>Why are you reformatting this?</div><div><br></div><div>-Eli</div><div><br></div><br><div class="gmail_extra">
<div class="gmail_quote">On Thu, Aug 15, 2013 at 6:20 AM, Kuperstein, Michael M <span dir="ltr"><<a href="mailto:michael.m.kuperstein@intel.com" target="_blank">michael.m.kuperstein@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">






<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class=""><span style="color:rgb(31,73,125)">Whoops, previous patch is bad.<u></u><u></u></span></p>
<p class=""><span style="color:rgb(31,73,125)">Attached is a hopefully saner one.<u></u><u></u></span></p>
<p class=""><span style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class=""><span style="color:rgb(31,73,125)">(Also adds a missing check for pow(2.0, x) -> exp2(x) *<b>correctly*</b>, that’s unrelated to the ldexp issue.)<u></u><u></u></span></p>
<p class=""><span style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<div>
<div style="border-style:solid none none;border-top-color:rgb(181,196,223);border-top-width:1pt;padding:3pt 0cm 0cm">
<p class="" style="line-height:normal"><b><span style="font-size:10pt;font-family:Tahoma,sans-serif">From:</span></b><span style="font-size:10pt;font-family:Tahoma,sans-serif"> Kuperstein, Michael M
<br>
<b>Sent:</b> Thursday, August 15, 2013 16:10<br>
<b>To:</b> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<b>Subject:</b> [PATCH] Fix pow simplification when fabs and sqrt are unavailable<u></u><u></u></span></p>
</div>
</div><div><div class="h5">
<p class=""><u></u> <u></u></p>
<p class="">Hi,<u></u><u></u></p>
<p class=""><u></u> <u></u></p>
<p class="">This adds a missing TLI check for the pow(x, 0.5) -> fabs(sqrt(x)) simplification.<u></u><u></u></p>
<p class=""><u></u> <u></u></p>
<p class="">There’s a similar problem with exp2, but I don’t see a LibFunc::ldexp I can check for.
<u></u><u></u></p>
<p class="">Anyone has any idea if it’s missing on purpose, or just an omission?
<u></u><u></u></p>
<p class=""><u></u> <u></u></p>
<p class="">Michael<u></u><u></u></p>
</div></div></div><div><div class="h5">
<p>---------------------------------------------------------------------<br>
Intel Israel (74) Limited</p>

<p>This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.</p></div></div></div>

<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>