G++-11销毁顺序由G++9更改

我们有以下代码(它更复杂,我试着做一个最小的例子)。

#include <iostream>
#include <vector>
#include <string>
#include <memory>

template<typename T>
struct use_type
{
    use_type(T& v) : value(v) {}
    T& value;
};

template<typename T>
use_type<T> use(T& value) { return use_type<T>(value); }

template<typename T>
use_type<T> use(const T& value) { return use_type<T>(const_cast<T&>(value)); }

template<typename T>
struct printer_helper
{
    void use(const use_type<T>& use) { uses.push_back(use); }
    void final_action()
    {
        for (const auto& use : uses)
        {
            std::cout << use.value.size() << std::endl;
            for (const auto& v : use.value)
            {
                std::cout << v << ",";
            }
            std::cout << std::endl;
        }
    }
    std::vector<use_type<T>> uses;
};

template<typename T>                                        
struct printer
{
    printer() { helper = new printer_helper<T>(); }
    printer<T>& operator , (const use_type<T>& t)
    {
        helper->use(t);
        return *this;
    }
    ~printer()
    {
        helper->final_action();
        delete helper;
    }
    printer(const printer&) = delete;
    printer& operator =(const printer&) = delete;
    printer(printer&&) = default;
    printer& operator =(printer&&) = default;
    printer_helper<T>* helper;
};

template<typename T>
struct printer_creator
{
    printer<T> operator << (const char*) { return {}; }
};

int main()
{
    using vec = std::vector<std::string>;
    {
    /*vec v1{"abc", "bcd"};
    vec v2{"new", "old", "real"};*/
    printer_creator<vec> p;
    p << "", use(vec{"abc", "bcd"}), use(vec{"new", "old", "real"});
    //p << p, use(v1), use(v2);
    }
}

这段代码在 g++-11 之前的 g++ 上工作得很好。使用 g++-11,它在输出中要么是段错误要么是垃圾。是UB吗?如果是的话,也许您可​​以告诉我,哪些 gcc 更改使其失败并在可能的情况下引用标准?

现场示例:

克++-10
克++-11

回答

临时向量在创建它们的行的末尾被破坏。p在其作用域结束时(即在向量之后)被破坏。由于p使用对现在悬空的向量的引用的析构函数,您的代码具有未定义的行为。

如果它曾经工作过,那纯粹是“运气”,向量析构函数使向量处于可用状态。


回答

p << "", use(vec{"abc", "bcd"}), use(vec{"new", "old", "real"});

你创建了 3 个临时工

  • printer<vec>来自p << "")
  • vec{"abc", "bcd"}
  • vec{"new", "old", "real"}

它们之间没有顺序(见下面的注释),所以它们可以按任何顺序创建。

只有从左到右的破坏是正确的(所以从右到左的构造),其他顺序将使用悬空引用并且是 UB。而且由于您没有订单保证......

注:
我们通过重载失去短路operator &&/ operator ||
我们失去了通过重载左到右的评估顺序operator,


以上是G++-11销毁顺序由G++9更改的全部内容。
THE END
分享
二维码
< <上一篇
下一篇>>