[compiler-rt] r357728 - Remove unneeded ymath.h include from int_math.h

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 4 14:47:16 PDT 2019


Author: rnk
Date: Thu Apr  4 14:47:15 2019
New Revision: 357728

URL: http://llvm.org/viewvc/llvm-project?rev=357728&view=rev
Log:
Remove unneeded ymath.h include from int_math.h

This avoids a conflict between stdbool.h, which defines bool to _Bool in
xkeycheck.h. From what I can tell, ymath.h is an internal header, and
the intention is that users should include math.h directly instead. It
doesn't appear to provide declarations of anything required for our
builtins. This include was added back in r249513 from 2015, and it's
possible that ymath.h provided something this code needed at the time,
but today it does not.

Modified:
    compiler-rt/trunk/lib/builtins/int_math.h

Modified: compiler-rt/trunk/lib/builtins/int_math.h
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/builtins/int_math.h?rev=357728&r1=357727&r2=357728&view=diff
==============================================================================
--- compiler-rt/trunk/lib/builtins/int_math.h (original)
+++ compiler-rt/trunk/lib/builtins/int_math.h Thu Apr  4 14:47:15 2019
@@ -27,7 +27,6 @@
 #if defined(_MSC_VER) && !defined(__clang__)
 #include <math.h>
 #include <stdlib.h>
-#include <ymath.h>
 #endif
 
 #if defined(_MSC_VER) && !defined(__clang__)




More information about the llvm-commits mailing list