[PATCH] D77776: [Driver] Drop support for FreeBSD < 10

Dimitry Andric via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 9 11:53:08 PDT 2020


dim added a comment.

In D77776#1971543 <https://reviews.llvm.org/D77776#1971543>, @arichardson wrote:

> Alternatively, the checks could be changed to also handle OSMajorVersion == 0 and translate that to 10.
>  This seems to be what NetBSD.cpp does. Darwin.cpp also infers the version from the host when running on macos.


Yeah, I think that is much better. If people want to explicitly target foobar-freebsd9, they should be free to do so. But the default should indeed be either the host version, or if that cannot be determined, the lowest supported version, which is currently 10. For these particular features the exact version doesn't matter that much though, it's just about whether libc++ should  be the default.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77776





More information about the cfe-commits mailing list