[PATCH] D43579: [libcxx] Do not include the C math.h header before __config

Peter Collingbourne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 21 11:08:30 PST 2018


pcc added inline comments.


================
Comment at: include/math.h:1499
+// has previously been included.
+#if defined(_LIBCPP_MSVCRT) && defined(_USE_MATH_DEFINES)
+#include_next <math.h>
----------------
Nit: you don't actually need the ` && defined(_USE_MATH_DEFINES)` part.


https://reviews.llvm.org/D43579





More information about the cfe-commits mailing list