yaxunl added inline comments.
================ Comment at: clang/test/Headers/Inputs/include/math.h:108-109 long lroundf(float __a); -int max(int __a, int __b); -int min(int __a, int __b); double modf(double __a, double *__b); ---------------- tra wrote: > Why were these functions removed? It does not seem related to the changes in > the patch? These functions caused failure in the added lit test. For C++, max/min are defined as templates in <algorithm>. There is no max/min in either standard C or C++ math.h. Their existence cause false alarms in lit tests. Removing them to be consistent with standard C/C++ headers. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150965/new/ https://reviews.llvm.org/D150965 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits