[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