[PATCH] D25440: [Target] remove TargetRecip class; move reciprocal estimate isel functionality to TargetLowering
Hal Finkel via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 17 11:40:17 PDT 2016
hfinkel added a comment.
I think this is an improvement. Can we go a bit further in order to further remove the boilerplate here and move the calls to getSqrtEnabled and, if not disabled, getSqrtRefinementSteps into the caller of getRsqrtEstimate (along with the analogous change for the Div estimates)? All of the implementations use 'VT = Operand.getValueType()'.
More information about the llvm-commits