[libcxx-commits] [PATCH] D124165: [libcxx][AIX] Enable ABI list checking for XCOFF

[David Tenty via Phabricator via libcxx-commits]

daltenty added a comment.

@ldionne It's worth noting what's in this patch right now won't be our final ABI lists unfortunately. Our build compiler currently doesn't implement visibility support, so `_LIBCPP_HIDE_FROM_ABI` and friends don't have the intended effect.  We're actively working on visibility for LLVM 15, but until that is all in we won't have a build compiler that can generate a stable ABI.

What's your preference for how we should handle this? If you're ok with a bit of churn in the lists till we stabilize I think this can land as is. If not we could potentially use a development clang for the CI with ABI checks on (once that's available).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124165