Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Is invocable and ambiguous call: bug in either g++ or clang

Consider the following code:

// Preamble
#include <iostream>
#include <type_traits>

// A base class
template <class T>
struct base {void operator()(T){};};

// Two derived classes inheriting from the same base classes
template <class... T>
struct derived1: base<T>... {using base<T>::operator()...;};
template <class... T>
struct derived2: base<T>... {using base<T>::operator()...;};

// A class inheriting from both derived1 and derived2
template <class T0, class... T>
struct functor: derived1<T0>, derived2<T0, T...> {
    using derived1<T0>::operator();
    using derived2<T0, T...>::operator();
};

// Main function
int main() {
    std::cout << std::is_invocable_v<functor<int, float, char>, int> << "\n";
    std::cout << std::is_invocable_v<functor<int, float, char>, float> << "\n";
    std::cout << std::is_invocable_v<functor<int, float, char>, char> << "\n";
    return 0;
}

The call to functor<int, float, char>::operator()(int) is ambiguous because this operator is inherited twice, from both derived1 and derived2 (and let's say that for convoluted SFINAE purpose I want it to be ambiguous).

On clang++-5.0, the output of the code is 0, 1, 1, while on g++-7.2 the output is 1, 1, 1. Which one is right? And would there be a workaround, creating a new struct is_unambiguously_invocable while waiting for the bugfix?

like image 580
Vincent Avatar asked Mar 03 '18 13:03

Vincent


1 Answers

Your reasoning is correct. Note that gcc correctly disallows the call itself:

functor<int, float, char>()(42); // error: base<int> is an ambiguous base

It just incorrectly detects that this invocation is ill-formed. Reported this as gcc bug 84869. T.C. added a further reduced reproduction in the bug report that does not have library dependencies:

struct base {
    void operator()(int ) { }
};

struct a : base { };
struct b : base { };

struct f: a, b {
    using a::operator();
    using b::operator();
};

template<class T> auto g(int) -> decltype(T()(0), 0);
template<class T> auto g(...) -> long;

template<class, class> struct Same;
template<class T> struct Same<T, T> {};

Same<decltype(g<f>(0)), long> s; // should be okay, but gcc errors because it
                                 // thinks decltype(g<f>(0)) is int
like image 149
Barry Avatar answered Sep 27 '22 23:09

Barry