[libcxx-commits] [PATCH] D69174: Refine check for `_LIBCPP_C_HAS_NO_GETS` on FreeBSD
Dimitry Andric via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Oct 18 12:40:03 PDT 2019
dim marked an inline comment as done.
dim added inline comments.
================
Comment at: include/__config:1169
+# if defined(_LIBCPP_MSVCRT) || \
+ (defined(__FreeBSD__) && __FreeBSD_version >= 1300043)
# define _LIBCPP_C_HAS_NO_GETS
----------------
ldionne wrote:
> Shouldn't this say `defined(__FreeBSD_version) && __FreeBSD_version >= 1300043`?
Good one; functionally it won't matter, since <osreldate.h>'s only function is to `#define` `__FreeBSD_version`, and it will always be defined. But I can see that it might be confusing to read.
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69174/new/
https://reviews.llvm.org/D69174
More information about the libcxx-commits
mailing list