[PATCH] D150397: [darwin] Declare _availability_version_check as weak_import instead of looking it up at runtime using dlsym

[Dan Field via Phabricator via llvm-commits]

dnfield added a comment.

I'm realizing I wasn't quite right on that.

Flutter _used_ to routinely build the dylib using some non-Xcode version of Clang for a while. We then switched to Xcode to make sure we could have compatible bitcode when building with an Xcode toolchain, but I think only for binaries that could end up being shipped to the store. Sometime since the store dropped bitcode support, we went back to using upstream Clang. This enables us avoid the need to support multiple Clang toolchains in our build, and in particular we can use a distributed compiler backend when using this version of Clang that we can't use when using Xcode.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150397