<div dir="ltr"><div>Thanks for checking that, Evandro. I see what's happening - on AArch64, we don't generate the generic N-R refinement because we have 'frsqrts'. So the 'Iterations' value that is guarding this codegen in generic DAGCombiner is '0'. AArch64 will need to duplicate more code, or (better) we should move the !Reciprocal portion of this sequence out where all targets can use it, and we'll kill the existing duplicated code from AArch64TargetLowering::getSqrtEstimate().<br><br></div>Let me see if the latter can be done cleanly. <br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 31, 2018 at 4:09 PM, Evandro Menezes <span dir="ltr"><<a href="mailto:e.menezes@samsung.com" target="_blank">e.menezes@samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <p><font size="-1">Hi, Sanjay.</font></p>
    <p><font size="-1">The patch LGTM, but, when trying on AArch64, I
        didn't see any difference in the AArch64 test case in attach. 
        I'm confused...</font></p>
    <pre class="m_2897925744413813936moz-signature" cols="72">-- 
Evandro Menezes
</pre><div><div class="h5">
    <div class="m_2897925744413813936moz-cite-prefix">On 01/26/2018 12:01 PM, Sanjay Patel
      via Phabricator via llvm-commits wrote:<br>
    </div>
    </div></div><blockquote type="cite">
      <pre><div><div class="h5">spatel added a reviewer: bkramer.
spatel added a comment.

Ping.

Note that on x86 Linux, specifying -fdenormal-fp-math=ieee doesn't inhibit linking crtfastmath.o ( <a class="m_2897925744413813936moz-txt-link-freetext" href="https://reviews.llvm.org/rL165240" target="_blank">https://reviews.llvm.org/<wbr>rL165240</a> / <a class="m_2897925744413813936moz-txt-link-freetext" href="https://bugs.llvm.org/show_bug.cgi?id=14024" target="_blank">https://bugs.llvm.org/show_<wbr>bug.cgi?id=14024</a> ), but I think that's a separate issue. (No matter what we do here in codegen, denormals will be off on that platform.)


<a class="m_2897925744413813936moz-txt-link-freetext" href="https://reviews.llvm.org/D42323" target="_blank">https://reviews.llvm.org/<wbr>D42323</a></div></div>



______________________________<wbr>_________________
llvm-commits mailing list
<a class="m_2897925744413813936moz-txt-link-abbreviated" href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>
<a class="m_2897925744413813936moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a>
</pre>
    </blockquote>
    <br>
  </div>

</blockquote></div><br></div>