ROCm/composable_kernel
1
0
Fork 0
mirror of https://github.com/ROCm/composable_kernel.git synced 2026-04-20 06:49:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Enable math defines for MSVC. (#3503)

Browse Source 
The symbol M_PI is breaking the build on Windows.  The _USE_MATH_DEFINES macro enables M_PI and other math constants on Windows. (I'm guessing this is more idomatic than the old trick of using PI=acos(-1.0).)

https://learn.microsoft.com/en-us/cpp/c-runtime-library/math-constants?view=msvc-170

Co-authored-by: BradPepersAMD <Brad.Pepers@amd.com>
This commit is contained in:
John Shumway
2026-01-02 11:36:42 -08:00
committed by GitHub
parent cc1392a405
commit 1da340031c
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
include/ck/library/utility/device_tensor_generator.hpp
View File

@@ -7,6 +7,7 @@
#include "ck/utility/common_header.hpp"
#include "ck/library/utility/device_tensor_generator.hpp"
#include "ck/utility/data_type.hpp"
#define _USE_MATH_DEFINES // Required for M_PI in MSVC
#include <cmath>
// use xorshift for now since it is simple. Should be suitable enough, but feel free to switch in