diff --git a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp index 2a546433a2454..d2ce11340a0f7 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp @@ -11,13 +11,6 @@ /// \todo This should be generated by TableGen. //===----------------------------------------------------------------------===// -#if defined(_MSC_VER) || defined(__MINGW32__) -// According to Microsoft, one must set _USE_MATH_DEFINES in order to get M_PI -// from the Visual C++ cmath / math.h headers: -// https://docs.microsoft.com/en-us/cpp/c-runtime-library/math-constants?view=vs-2019 -#define _USE_MATH_DEFINES -#endif - #include "AMDGPULegalizerInfo.h" #include "AMDGPU.h" @@ -1877,7 +1870,7 @@ bool AMDGPULegalizerInfo::legalizeSinCos( unsigned Flags = MI.getFlags(); Register TrigVal; - auto OneOver2Pi = B.buildFConstant(Ty, 0.5 / M_PI); + auto OneOver2Pi = B.buildFConstant(Ty, 0.5 / numbers::pi); if (ST.hasTrigReducedRange()) { auto MulVal = B.buildFMul(Ty, SrcReg, OneOver2Pi, Flags); TrigVal = B.buildIntrinsic(Intrinsic::amdgcn_fract, {Ty}, false) diff --git a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp index 3b8930c433a3a..ace28d5c9dcb2 100644 --- a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp +++ b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp @@ -11,11 +11,6 @@ // //===----------------------------------------------------------------------===// -#if defined(_MSC_VER) || defined(__MINGW32__) -// Provide M_PI. -#define _USE_MATH_DEFINES -#endif - #include "SIISelLowering.h" #include "AMDGPU.h" #include "AMDGPUSubtarget.h" @@ -8230,7 +8225,7 @@ SDValue SITargetLowering::LowerTrig(SDValue Op, SelectionDAG &DAG) const { // TODO: Should this propagate fast-math-flags? - SDValue OneOver2Pi = DAG.getConstantFP(0.5 / M_PI, DL, VT); + SDValue OneOver2Pi = DAG.getConstantFP(0.5 / numbers::pi, DL, VT); if (Subtarget->hasTrigReducedRange()) { SDValue MulVal = DAG.getNode(ISD::FMUL, DL, VT, Arg, OneOver2Pi);