[PATCH] D25440: [Target] remove TargetRecip class; move reciprocal estimate isel functionality to TargetLowering

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 10:45:53 PDT 2016

echristo added a comment.

I had one inline bikeshed type of comment, but otherwise also LGTM. Thanks for doing this work!


Comment at: include/llvm/Target/TargetLowering.h:260-266
+  int getSqrtEnabled(EVT VT, MachineFunction &MF) const;
+  /// Return a ReciprocalEstimate enum value for a division of the given type
+  /// based on the function's attributes. If the operation is not overridden by
+  /// the function's attributes, "Unspecified" is returned and target defaults
+  /// are expected to be used for instruction selection.
+  int getDivEnabled(EVT VT, MachineFunction &MF) const;
Bit of a bikeshed but these two functions sound like they're seeing if sqrt and div are instructions in the hardware.



More information about the llvm-commits mailing list