[clang] [Clang] Remove __is_referenceable builtin (PR #123078)
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 15 11:07:49 PST 2025
erichkeane wrote:
> Should we deprecate it in 20, remove in 21? _almost_ is not 0 https://github.com/search?q=__is_referenceable%28&type=code
Eh, it is pretty effectively 0. If you subtract ones that are wrapped in '#if 0' or part of libcxx, the first two pages have like 3 entries on them.
> > Should we deprecate it in 20, remove in 21? _almost_ is not 0 https://github.com/search?q=__is_referenceable%28&type=code
>
> What exactly would you like to do? Make `__has_builtin(__is_referenceable)` return 0 but still accept `__is_referenceable` for a release?
I would just say 'document for a release' then remove, but that is a better idea. perhaps `__has_builtin` is 0, plus some sort of `__HAD_DEPRECATED_IS_REFERENCEABLE` macro?
https://github.com/llvm/llvm-project/pull/123078
More information about the cfe-commits
mailing list