diff options
| author | Nathaniel Shead <nathanieloshead@gmail.com> | 2024-07-08 22:25:17 +1000 |
|---|---|---|
| committer | Nathaniel Shead <nathanieloshead@gmail.com> | 2024-07-17 11:21:58 +1000 |
| commit | 1aa0f1627857c3e2d90982bdb07ca78ca10b26f3 (patch) | |
| tree | 9036f0581f412fef640e346dbbeb70725f11dbad /gcc/cp/name-lookup.cc | |
| parent | 72bce1fbef6b03e875501dd953dc8cc40f395d31 (diff) | |
| download | gcc-1aa0f1627857c3e2d90982bdb07ca78ca10b26f3.zip gcc-1aa0f1627857c3e2d90982bdb07ca78ca10b26f3.tar.gz gcc-1aa0f1627857c3e2d90982bdb07ca78ca10b26f3.tar.bz2 | |
c++/modules: Propagate BINDING_VECTOR_*_DUPS_P on realloc [PR99242]
When importing modules, when a binding vector for a name runs out of
slots it gets reallocated with a larger size, and existing bindings are
copied across. However, the flags to indicate whether deduping needs to
occur did not: this causes ICEs, as it allows a duplicate binding to be
added which then violates assumptions later on.
PR c++/99242
gcc/cp/ChangeLog:
* name-lookup.cc (append_imported_binding_slot): Propagate dups
flags.
gcc/testsuite/ChangeLog:
* g++.dg/modules/pr99242_a.H: New test.
* g++.dg/modules/pr99242_b.H: New test.
* g++.dg/modules/pr99242_c.H: New test.
* g++.dg/modules/pr99242_d.C: New test.
Signed-off-by: Nathaniel Shead <nathanieloshead@gmail.com>
Diffstat (limited to 'gcc/cp/name-lookup.cc')
| -rw-r--r-- | gcc/cp/name-lookup.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/gcc/cp/name-lookup.cc b/gcc/cp/name-lookup.cc index 361dc3d..8823ab7 100644 --- a/gcc/cp/name-lookup.cc +++ b/gcc/cp/name-lookup.cc @@ -353,6 +353,10 @@ append_imported_binding_slot (tree *slot, tree name, unsigned ix) tree new_vec = make_binding_vec (name, want); BINDING_VECTOR_NUM_CLUSTERS (new_vec) = have + 1; + BINDING_VECTOR_GLOBAL_DUPS_P (new_vec) + = BINDING_VECTOR_GLOBAL_DUPS_P (*slot); + BINDING_VECTOR_PARTITION_DUPS_P (new_vec) + = BINDING_VECTOR_PARTITION_DUPS_P (*slot); memcpy (BINDING_VECTOR_CLUSTER_BASE (new_vec), BINDING_VECTOR_CLUSTER_BASE (*slot), have * sizeof (binding_cluster)); |