[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