CODEWITHSUNDEEP
c++visual-c++lambdaclangcopy-constructor
JohnB
JohnB

Reputation: 13723

VC++ 15 calls the wrong copy constructor for lambda capture?

Consider the following program:

#include <iostream>
struct X {
  X () = default;
  X (X &) { std::cout << "non-const called" << std::endl; }
  X (X const &) { std::cout << "const called" << std::endl; }
  int i () const { return 7; }
};

auto f () {
  X x;
  auto lambda = [=]() { return x.i(); };
  return lambda;
}

int main()
{
  auto lambda = f();
  std::cout << lambda () << std::endl;
  return 0;
}

With VC++15, I get the output

const called
const called
7

With Clang 3.9, I get

non-const called
7

Which compiler is correct here?

Upvotes: 13

Views: 256

Answers (1)

skypjack
skypjack

Reputation: 50550

I would say that clang is right.
The most suited constructor is called only once when the lambda captures x and the constructor for the returned value is optimized out.
That's why you obtain only one non-const called.

See here and here for further details about copy-elision and RVO.

Upvotes: 2

Related Questions