https://gcc.gnu.org/g:57113e3db168e19c46ca52ad084790d462eea603
commit r15-1295-g57113e3db168e19c46ca52ad084790d462eea603 Author: Jason Merrill <ja...@redhat.com> Date: Wed Jun 12 21:43:45 2024 -0400 c++: adjust comment Adjusting the comment I added in r15-1223 to clarify that this is a workaround for a bug elsewhere. gcc/cp/ChangeLog: * module.cc (depset::hash::add_binding_entity): Adjust comment. Diff: --- gcc/cp/module.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/gcc/cp/module.cc b/gcc/cp/module.cc index 72e876cec187..ea7ad0c1f291 100644 --- a/gcc/cp/module.cc +++ b/gcc/cp/module.cc @@ -13254,9 +13254,10 @@ depset::hash::add_binding_entity (tree decl, WMB_Flags flags, void *data_) if (data->hash->add_namespace_entities (decl, data->partitions)) { /* It contains an exported thing, so it is exported. - We used to assert DECL_MODULE_PURVIEW_P, but that fails for a - namespace like std::__exception_ptr which is never opened in - module purview; the exporting using finds another using. */ + + FIXME we have to set DECL_MODULE_PURVIEW_P instead of asserting + that it is already set because of the c++/114683 issue with + exported using-declarations; see do_nonmember_using_decl. */ DECL_MODULE_PURVIEW_P (decl) = true; DECL_MODULE_EXPORT_P (decl) = true; }