Added flag to enable _USE_MATH_DEFINES on windows

Windows builds were flaking because they were unable to see the `M_PI` symbol. This will add the `_USE_MATH_DEFINES` flag into our builds which should hopefully fix the issue.
hongyunnchen · Mar 12, 2020 · 396ffa1 · 396ffa1
1 parent a0ef76b
commit 396ffa1
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/tensorflow_quantum/core/ops/BUILD b/tensorflow_quantum/core/ops/BUILD
@@ -20,6 +20,7 @@ cc_binary(
     copts = select({
         ":windows": [
             "/D__CLANG_SUPPORT_DYN_ANNOTATION__",
+            "/D_USE_MATH_DEFINES",
             "/DEIGEN_MPL2_ONLY",
             "/DEIGEN_MAX_ALIGN_BYTES=64",
             "/DEIGEN_HAS_TYPE_TRAITS=0",
@@ -74,6 +75,7 @@ cc_binary(
     copts = select({
         ":windows": [
             "/D__CLANG_SUPPORT_DYN_ANNOTATION__",
+            "/D_USE_MATH_DEFINES",
             "/DEIGEN_MPL2_ONLY",
             "/DEIGEN_MAX_ALIGN_BYTES=64",
             "/DEIGEN_HAS_TYPE_TRAITS=0",
@@ -134,6 +136,7 @@ cc_binary(
     copts = select({
         ":windows": [
             "/D__CLANG_SUPPORT_DYN_ANNOTATION__",
+            "/D_USE_MATH_DEFINES",
             "/DEIGEN_MPL2_ONLY",
             "/DEIGEN_MAX_ALIGN_BYTES=64",
             "/DEIGEN_HAS_TYPE_TRAITS=0",