[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()'.


https://reviews.llvm.org/D25440





More information about the llvm-commits mailing list