[PATCH] D61765: [OpenMP][Clang][BugFix] Split declares and math functions inclusion.
Johannes Doerfert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 10 12:40:38 PDT 2019
jdoerfert added a comment.
What about `abs` tests?
================
Comment at: lib/Headers/openmp_wrappers/__clang_openmp_math_declares.h:17
+ #include <cstdlib>
+ #include <stdlib.h>
+#endif
----------------
Why do we need the stdlib includes again?
================
Comment at: test/Headers/Inputs/include/cstdlib:2
+#pragma once
+typedef __SIZE_TYPE__ size_t;
----------------
Where is this used? Are there tests missing?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61765/new/
https://reviews.llvm.org/D61765
More information about the cfe-commits
mailing list