У меня есть следующий код
https://godbolt.org/z/nehcPdxGe
#include <boost/asio.hpp>
#include <boost/asio/awaitable.hpp>
#include <boost/asio/co_spawn.hpp>
#include <gtest/gtest.h>
template <typename FUNC, typename EXECUTOR> void foo(FUNC, EXECUTOR) {}
struct Marker {
void const* ptr;
Marker() : ptr(this) {}
Marker(Marker const& rhs) = delete;
~Marker() { EXPECT_EQ(this, ptr); }
};
boost::asio::awaitable<int> xxx() { co_return 0; }
boost::asio::awaitable<void> test_in_coro() {
foo([marker = Marker{}]() -> int { return 42; }, co_await xxx());
co_return;
}
TEST(BugInCoro, Reproduce) {
boost::asio::io_context ioc;
boost::asio::co_spawn(ioc, test_in_coro(), boost::asio::detached);
ioc.run();
}
int main() {
testing::InitGoogleTest();
return RUN_ALL_TESTS();
}
Утверждение в деструкторе Marker
не выполняется. Более того, я проверил, что ptr
такой же, как в конструкторе, а this
отличается.
Все работает нормально, если я просыпаюсь перед вызовом foo
или сохраняю лямбду в локальной переменной.
Это происходит в gcc 11.x и 12.x, но не в 13.x и не в clang.
Я впадаю в какое-то неопределенное поведение? Или, может быть, это ошибка компилятора?
Пожалуйста, отредактируйте свой код, чтобы предоставить минимально воспроизводимый пример. На самом деле вы повторили свой код.
Исправлен автономный пример. Интересно, что на моей машине он работает корректно (тот же буст, тот же gcc): imgur.com/FR61Jkg
Да, это проблема компилятора. Это было исправлено в GCC13.
Насколько я могу судить, это было исправлено в https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=58a7b1e354530d8dfe7d8fb859c8b8b5a9140f1f
coroutines: Do not promote temporaries that will be elided.
We usually need to 'promote' (i.e. save to the coroutine frame) any temporary
variable that is in a target expression that must persist across an await
expression. However, if the TE is just used as a direct initializer for
another object it will be elided - and we should not promote it since that
would lead to a DTOR call for something that is never constructed.
Since we now have a mechanism to tell if TEs will be elided, use that.
Although the PRs referenced initially appear to be different issues, they all
stem from this.
Co-Authored-By: Adrian Perl <[email protected]>
Signed-off-by: Iain Sandoe <[email protected]>
PR c++/100611
PR c++/101367
PR c++/101976
PR c++/99576
gcc/cp/ChangeLog:
* coroutines.cc (find_interesting_subtree): Do not promote temporaries
that are only used as direct initializers for some other object.
gcc/testsuite/ChangeLog:
* g++.dg/coroutines/pr100611.C: New test.
* g++.dg/coroutines/pr101367.C: New test.
* g++.dg/coroutines/pr101976.C: New test.
* g++.dg/coroutines/pr99576_1.C: New test.
* g++.dg/coroutines/pr99576_2.C: New test.
Вы можете просмотреть любую из упомянутых ссылок на ошибки для получения дополнительной информации и увидеть, что некоторые из воспроизводителей в основном идентичны вашим, за исключением того, что они не используют типы обещаний Asio.
это один и тот же код, опубликованный дважды?