[PATCH] Use the 'arcp' fast-math-flag when combining repeated FP divisors

Sanjay Patel spatel at rotateright.com
Tue May 12 13:04:32 PDT 2015


In http://reviews.llvm.org/D9708#171355, @ahatanak wrote:

> 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?).


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.

Is a separate mechanism needed to handle the llc command-line flag "-enable-unsafe-fp-math"?


http://reviews.llvm.org/D9708

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list