[PATCH] D61949: [OpenMP][bugfix] Fix issues with C++ 17 compilation when handling math functions

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 15 12:15:34 PDT 2019


jdoerfert requested changes to this revision.
jdoerfert added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Headers/openmp_wrappers/__clang_openmp_math_declares.h:24
+  #include <cstdlib>
+  #include <stdlib.h>
 #endif
----------------
jdoerfert wrote:
> I ask this question again and again, do we need them now? Why?
Why did you add these includes?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61949/new/

https://reviews.llvm.org/D61949





More information about the cfe-commits mailing list