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

[Petr Hosek via Phabricator via llvm-commits]

phosek added a comment.

After this change, Apple App Store is rejecting Flutter apps with the following error message:

  ITMS-90338: Non-public API usage - The app references non-public symbols in Frameworks/Flutter.framework/Flutter: __availability_version_check

I can confirm that this change results in undefined `__availability_version_check` symbol:

  $ nm Flutter.xcframework/ios-arm64/Flutter.framework/Flutter | grep __availability_version_check
                   U __availability_version_check

Is that intentional? Does App Store need to be updated? Would it be possible to revert this change in the meantime?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150397