[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