What you are doing is technically UB and is allowed to fail — there is no guarantee that using pointers to member functions of standard library classes like this actually works. For libc++, it doesn't.
Concretely, the issue is given a hierarchy like:
struct B { bool has_value() const; }
struct D : B { };
The type of &B::has_value
is obviously a bool (B::*)() const
. But the type of &D::has_value
... is also that same thing. Personally I think that's a language defect and it should give you a bool (D::*)() const
since that's what you asked for — but that's very likely not changeable right now, and them's the rules.
Now, for libstdc++ and MSVCSTL, &std::optional<int>::has_value
gives you a bool (std::optional<int>::*)() const
because they apparently implement that member function directly. But for libc++, they apparently implement their optional
a little but differently... so the member point your get back is actually a bool (std::__optional_storage_base<int>::*)() const
. Well, also noexcept
, but that doesn't matter.
Now you might think this doesn't matter — after all, you can invoke base class member functions fine, right? You can. Unless it's a private base. Which, in this case, it is.
In reduced form, libstdc++ and MSVCSTL look like this:
template <class T> struct optional { bool has_value() const; };
While libc++ looks like this:
template <class T> struct optional_base { bool has_value() const; };
template <class T> struct optional : private optional_base<T> {
using optional_base<T>::has_value;
};
The result is that while o.has_value()
works for all the implementations, attempting to use &optional<int>::has_value
for libc++ isn't invocable because you get a pointer to a private base class function.
Incidentally, one of the reasons that it would be great if pointers-to-members were invocable is the quality of error messages.
Consider:
template <class F, class T>
concept invocable = requires (F f, T t) {
#if DIRECT
(t.*f)();
#else
std::invoke(f, t);
#endif
};
static_assert(invocable<decltype(&std::optional<int>::has_value), std::optional<int>>);
On clang, this concept fails either way, since it's checking the same thing either way. But the quality of error is quite a bit different. With std::invoke
:
<source>:7:5: note: because 'std::invoke(f, t)' would be invalid: no matching function for call to 'invoke'
7 | std::invoke(f, t);
| ^
With (t.*f)()
:
<source>:7:7: note: because '(t .* f)()' would be invalid: cannot cast 'std::optional<int>' to its private base class 'std::__optional_storage_base<int>'
7 | (t.*f)();
| ^
gcc's disparity is similar (although you have to do fconcepts-diagnostics-depth=2
). With the direct invocation, you get a diagnostic about the base class being inaccessible. With invoke
you get... nothing.
Which would you rather see?