maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Remove math.h/cmath include from DataTypes.h 

DataTypes.h is meant to wrap the integer type and limits headers, which
have some unfortunate variance. The FP math functions declared by math.h
are not unnecessary. math.h took a noticeable amount of time to parse
(~40ms), but that could be startup costs.

Anyway, we don't need to include it, so skipping it can't hurt.

This has been present since the initial CMake build was added in 2008.
This commit is contained in:
Reid Kleckner 2020-04-06 11:14:59 -07:00
parent 9ed0612cca
commit 3a29393b47
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/include/llvm-c/DataTypes.h
View File

@ -24,12 +24,6 @@
#ifndef LLVM_C_DATATYPES_H #ifndef LLVM_C_DATATYPES_H
#define LLVM_C_DATATYPES_H #define LLVM_C_DATATYPES_H
#ifdef __cplusplus
#include <cmath>
#else
#include <math.h>
#endif
#include <inttypes.h> #include <inttypes.h>
#include <stdint.h> #include <stdint.h>