[PATCH] Use the 'arcp' fast-math-flag when combining repeated FP divisors
Sanjay Patel
spatel at rotateright.com
Tue May 12 10:15:30 PDT 2015
Hi hfinkel, echristo, ab, kariddi, milseman,
This is a usage of the IR-level fast-math-flags now that they are propagated to SDNodes (r237046). This was originally part of D8900.
I'm not sure if we need to maintain the global TargetOptions.UnsafeFPMath checks or if we're now free to start deleting those and rely exclusively on the node-level optimization flags. If the latter, I need to update the corresponding fdiv testcases for PowerPC and Aarch64 to use 'arcp'.
http://reviews.llvm.org/D9708
Files:
lib/CodeGen/SelectionDAG/DAGCombiner.cpp
test/CodeGen/X86/fdiv-combine.ll
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9708.25597.patch
Type: text/x-patch
Size: 4482 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150512/24251106/attachment.bin>
More information about the llvm-commits
mailing list