g ++ 5.2 - 使用整数文字调用模板参数包时未初始化警告

g++ 5.2 -Wuninitialized warning when calling template parameter pack with integer literal

看到this Meeting C++ 2015 lightning talk后,我试验了这段代码

#include <iostream>
#include <utility>

template <typename Fun, typename... Param>
auto generic_bind(Fun fun, Param&&... param)
{
    return [fun, &param...] (auto&&... x)
    {
        return fun(std::forward<Param>(param)... , 
                   std::forward<decltype(x)>(x)...);
    };  
}

int demo(int a, int b, int c) { return a * b + c; }

int main()
{
    // does work with int variables
    int a = 11, b = 22, c = 33;
    auto f1 = generic_bind(demo, a); 
    auto f2 = generic_bind(f1, b);
    std::cout << f1(b, c) << f2(33); // both result in 275
}

main() 正文更改为

     // does not work with int literals in g++ 5.2.0
     auto f1 = generic_bind(demo, 11); 
     auto f2 = generic_bind(f1, 22);
     std::cout << f1(22, 33) << f2(33);

clang++ -Wall 都工作正常,但 GNU g++ 5.2 创建了一个 -Wuninitialized 警告:

main.cpp: In function 'int main()':
main.cpp:14:42: warning: '<anonymous>' is used uninitialized in this function [-Wuninitialized]
int demo(int a, int b, int c) { return a * b + c; }
                                         ^
main.cpp:27:31: note: '<anonymous>' was declared here
  auto f2 = generic_bind(f1, 22);
                               ^
main.cpp:14:42: warning: '<anonymous>' is used uninitialized in this function [-Wuninitialized]
int demo(int a, int b, int c) { return a * b + c; }
                                         ^
main.cpp:26:36: note: '<anonymous>' was declared here
     auto f1 = generic_bind(demo, 11); 

并给出意外结果 33(参见:Coliru live example)。哪个编译器在 C++14 上是正确的?

程序有未定义的行为,所以任何事情都有可能发生,所以可以说两个编译器都是正确的! GCC 的警告是存在问题的有用线索,即使它们没有解释确切的问题。

当您使用整数文字时,它会导致创建临时 int 对象,并且 Params&& 引用参数绑定到这些临时右值。然后,您的 lambda 使用引用捕获,因此您 return 的闭包包含对临时对象的引用。

那些临时变量在完整表达式的末尾超出范围,即在调用 generic_bind 之后的分号处。这意味着当您调用 f1f2 时,您从悬空引用中读取,这是未定义的行为。

在原始代码中,Params&& 参数绑定到自动变量 abc,然后闭包包含对这些相同对象的引用,并且当您调用 f1f2 时它们仍在范围内。所以原始代码是可以的(尽管如果 f1f2 转义到更广泛的范围并且比 abc 长,你会得到相同的问题)。