<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 12, 2015 at 1:04 PM, Sanjay Patel <span dir="ltr"><<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>In <a href="http://reviews.llvm.org/D9708#171355" target="_blank">http://reviews.llvm.org/D9708#171355</a>, @ahatanak wrote:<br>
<br>
> To preserve backward compatibility, I think we should make sure functions using fast math attributes (e.g., "unsafe-fp-math" in test/CodeGen/PowerPC/fdiv-combine.ll) still get optimized after we migrate to modeling those flags at the node-level (using autoupgrade?).<br>
<br>
<br>
</span>Thanks, Akira. If I understand correctly, I should create this autoupgrade patch first then. That would allow us to remove the Options.UnsafeFPMath checks as we update all of the places in the backend to use the node-level flags.<br>
<br></blockquote><div><br></div><div>The autoupgrade patch will be needed when UnsafeFPMath is removed, but I'm not sure removing it completely should be a prerequisite to committing this patch. If it turns out to be not trivial to remove it completely, it might be acceptable to do it later.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is a separate mechanism needed to handle the llc command-line flag "-enable-unsafe-fp-math"?<br>
<div><div><br></div></div></blockquote><div><br></div><div>If we decide to keep the command line option, yes, we have to fix the handling of the flag.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
<br>
<a href="http://reviews.llvm.org/D9708" target="_blank">http://reviews.llvm.org/D9708</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>