[PATCH] [AArch64] Combine multiple FDIVs with the same divisor
Sanjay Patel
spatel at rotateright.com
Thu Nov 20 08:02:12 PST 2014
Just chiming in for the target-independent path: x86 needs this too.
I don't understand the expense argument cited here:
http://llvm.org/bugs/show_bug.cgi?id=16218#c4
Is checking uses in InstCombine more expensive than in DAGCombine? This transform is only firing with fast FP-math. Does that make it any more tolerable?
http://reviews.llvm.org/D6334
More information about the llvm-commits
mailing list