[llvm] [x86] Add tan intrinsic part 4 (PR #90503)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 15:11:47 PDT 2024


================
@@ -729,12 +731,14 @@ X86TargetLowering::X86TargetLowering(const X86TargetMachine &TM,
     setOperationAction(ISD::FSIN   , MVT::f32, Expand);
     setOperationAction(ISD::FCOS   , MVT::f32, Expand);
     setOperationAction(ISD::FSINCOS, MVT::f32, Expand);
+    setOperationAction(ISD::FTAN, MVT::f32, Expand);
 
     if (UseX87) {
       // Always expand sin/cos functions even though x87 has an instruction.
       setOperationAction(ISD::FSIN, MVT::f64, Expand);
       setOperationAction(ISD::FCOS, MVT::f64, Expand);
       setOperationAction(ISD::FSINCOS, MVT::f64, Expand);
+      setOperationAction(ISD::FTAN, MVT::f64, Expand);
----------------
topperc wrote:

I think this should be unneeded due to TargetLoweringBase.cpp defaulting to Expand.

https://github.com/llvm/llvm-project/pull/90503


More information about the llvm-commits mailing list