]> git.ipfire.org Git - thirdparty/gcc.git/commit
c++/modules: inline loaded at eof
authorJason Merrill <jason@redhat.com>
Tue, 1 Apr 2025 23:22:18 +0000 (19:22 -0400)
committerJason Merrill <jason@redhat.com>
Thu, 3 Apr 2025 15:19:30 +0000 (11:19 -0400)
commitacfe19f03bccd1f3183fd5d70d7a3c8c85e62fac
treedf1fade65c7da69b2569565ef19eb845dc82c104
parentbd5597156ca0c7d6fb50c6fe92a7abe6717cb2b5
c++/modules: inline loaded at eof

std/format/string.cc and a few other libstdc++ tests were failing with
module std with undefined references to __failed_to_parse_format_spec.  This
turned out to be because since r15-8012 we don't end up calling
note_vague_linkage_fn for functions loaded after at_eof is set.

But once import_export_decl decides on COMDAT linkage, we should be able to
just clear DECL_EXTERNAL and let cgraph take it from there.

I initially made this change in import_export_decl, but decided that for GCC
15 it would be safer to limit the change to modules.  For GCC 16 I'd like to
do away with DECL_NOT_REALLY_EXTERN entirely, it's been obsolete since
cgraphunit in 2003.

gcc/cp/ChangeLog:

* module.cc (module_state::read_cluster)
(post_load_processing): Clear DECL_EXTERNAL if DECL_COMDAT.
gcc/cp/module.cc