diff options
author | Patrick Palka <ppalka@redhat.com> | 2024-07-22 21:30:49 -0400 |
---|---|---|
committer | Patrick Palka <ppalka@redhat.com> | 2024-07-22 21:32:07 -0400 |
commit | 066c7893eae0bfc7d9b33b931f115f455246c914 (patch) | |
tree | 4b07036fe945ab11c41fe4e214f38298bfabcaa2 /libjava/win32-threads.cc | |
parent | 50ff112d17ec53ccff2858f6ded9dc04b1d5d2bc (diff) | |
download | gcc-releases/gcc-14.zip gcc-releases/gcc-14.tar.gz gcc-releases/gcc-14.tar.bz2 |
c++/coroutines: correct passing *this to promise type [PR104981]releases/gcc-14
When passing *this to the promise type ctor (or to its operator new)
(as per [dcl.fct.def.coroutine]/4), we add an explicit cast to lvalue
reference. But this is unnecessary since *this is already always an
lvalue. And doing so means we need to call convert_from_reference
afterward to lower the reference expression to an implicit dereference,
which we're currently neglecting to do and which causes overload
resolution to get confused when computing argument conversions.
So this patch removes this unneeded reference cast when passing *this
to the promise ctor, and removes both the cast and implicit deref when
passing *this to operator new, for consistency. While we're here, use
cp_build_fold_indirect_ref instead of directly building INDIRECT_REF.
PR c++/104981
PR c++/115550
gcc/cp/ChangeLog:
* coroutines.cc (morph_fn_to_coro): Remove unneeded calls
to convert_to_reference and convert_from_reference when
passing *this. Use cp_build_fold_indirect_ref instead
of directly building INDIRECT_REF.
gcc/testsuite/ChangeLog:
* g++.dg/coroutines/pr104981-preview-this.C: New test.
* g++.dg/coroutines/pr115550-preview-this.C: New test.
Reviewed-by: Iain Sandoe <iain@sandoe.co.uk>
Reviewed-by: Jason Merrill <jason@redhat.com>
(cherry picked from commit 7c5a9bf1d206fe20cb050200d4a30f11c76b1b19)
Diffstat (limited to 'libjava/win32-threads.cc')
0 files changed, 0 insertions, 0 deletions