diff options
| author | Iain Sandoe <iain@sandoe.co.uk> | 2023-03-30 13:14:23 +0530 |
|---|---|---|
| committer | Iain Sandoe <iain@sandoe.co.uk> | 2023-04-01 09:27:15 +0530 |
| commit | fc4cde2e6aa4d6ebdf7f70b7b4359fb59a1915ae (patch) | |
| tree | 10e6bf37e841f7704ce98d5ac374432d0015b558 | |
| parent | e11cde0381c528323abd0d57ed11ac8dae56ae31 (diff) | |
| download | gcc-fc4cde2e6aa4d6ebdf7f70b7b4359fb59a1915ae.zip gcc-fc4cde2e6aa4d6ebdf7f70b7b4359fb59a1915ae.tar.gz gcc-fc4cde2e6aa4d6ebdf7f70b7b4359fb59a1915ae.tar.bz2 | |
c++,coroutines: Stabilize names of promoted slot vars [PR101118].
When we need to 'promote' a value (i.e. store it in the coroutine frame) it
is given a frame entry name. This was based on the DECL_UID for slot vars.
However, when LTO is used, the names from multiple TUs become visible at the
same time, and the DECL_UIDs usually differ between units. This leads to a
"ODR mismatch" warning for the frame type.
The fix here is to use the current promoted temporaries count to produce
the name, this is stable between TUs and computed per coroutine.
Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>
PR c++/101118
gcc/cp/ChangeLog:
* coroutines.cc (flatten_await_stmt): Use the current count of
promoted temporaries to build a unique name for the frame entries.
| -rw-r--r-- | gcc/cp/coroutines.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gcc/cp/coroutines.cc b/gcc/cp/coroutines.cc index a2189e4..b307c8c 100644 --- a/gcc/cp/coroutines.cc +++ b/gcc/cp/coroutines.cc @@ -2889,7 +2889,7 @@ flatten_await_stmt (var_nest_node *n, hash_set<tree> *promoted, tree init = t; temps_used->add (init); tree var_type = TREE_TYPE (init); - char *buf = xasprintf ("D.%d", DECL_UID (TREE_OPERAND (init, 0))); + char *buf = xasprintf ("T%03u", (unsigned) temps_used->elements ()); tree var = build_lang_decl (VAR_DECL, get_identifier (buf), var_type); DECL_ARTIFICIAL (var) = true; free (buf); |