Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Curiously recurring template pattern (CRTP) with static constexpr in Clang

Consider my simple example below:

#include <iostream>

template <typename T>
class Base
{
public:
    static constexpr int y = T::x;
};

class Derived : public Base<Derived>
{
public:
    static constexpr int x = 5;
};


int main()
{
    std::cout << Derived::y << std::endl;
}

In g++, this compiles fine and prints 5 as expected. In Clang, however, it fails to compile with the error no member named 'x' in 'Derived'. As far as I can tell this is correct code. Is there something wrong with what I am doing, and if not, is there a way to have this work in Clang?

like image 300
Caleb Zulawski Avatar asked Mar 07 '16 21:03

Caleb Zulawski


Video Answer


2 Answers

This probably isn't the answer anyone would be looking for, but I solved the problem by adding a third class:

#include <iostream>

template <typename T>
class Base
{
public:
    static constexpr int y = T::x;
};

class Data
{
public:
     static constexpr int x = 5;
};

class Derived : public Base<Data>, public Data {};

int main()
{
    std::cout << Derived::y << std::endl;
}

It works as desired, but unfortunately it doesn't really have the benefits of CRTP!

like image 169
Caleb Zulawski Avatar answered Oct 28 '22 03:10

Caleb Zulawski


As linked in the comments, Initializing a static constexpr data member of the base class by using a static constexpr data member of the derived class suggests that clang behaviour is standard conformant up to C++14 here. Starting with Clang 3.9, your code compiles successfully with -std=c++1z. An easy possible workaround is to use constexpr functions instead of values:

#include <iostream>

template <typename T>
class Base
{
public:
    static constexpr int y() {return T::x();}
};

class Derived : public Base<Derived>
{
public:
    static constexpr int x() {return 5;}
};

int main()
{
    std::cout << Derived::y() << std::endl;
}
like image 25
user1531083 Avatar answered Oct 28 '22 03:10

user1531083