[Bug c++/113708] [modules] ICE in import_export_decl with non-trivially initialized inline variable in header module

["cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113708

--- Comment #1 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Nathaniel Shead <nshead@gcc.gnu.org>:

https://gcc.gnu.org/g:dd9d14f7d53de07beff06004922a2bff20ece671

commit r14-8987-gdd9d14f7d53de07beff06004922a2bff20ece671
Author: Nathaniel Shead <nathanieloshead@gmail.com>
Date:   Wed Feb 14 12:26:03 2024 +1100

    c++: Defer emitting inline variables [PR113708]

    Inline variables are vague-linkage, and may or may not need to be
    emitted in any TU that they are part of, similarly to e.g. template
    instantiations.

    Currently 'import_export_decl' assumes that inline variables have
    already been emitted when it comes to end-of-TU processing, and so
    crashes when importing non-trivially-initialised variables from a
    module, as they have not yet been finalised.

    This patch fixes this by ensuring that inline variables are always
    deferred till end-of-TU processing, unifying the behaviour for module
    and non-module code.

            PR c++/113708

    gcc/cp/ChangeLog:

            * decl.cc (make_rtl_for_nonlocal_decl): Defer inline variables.
            * decl2.cc (import_export_decl): Support inline variables.

    gcc/testsuite/ChangeLog:

            * g++.dg/debug/dwarf2/inline-var-1.C: Reference 'a' to ensure it
            is emitted.
            * g++.dg/debug/dwarf2/inline-var-3.C: Likewise.
            * g++.dg/modules/init-7_a.H: New test.
            * g++.dg/modules/init-7_b.C: New test.

    Signed-off-by: Nathaniel Shead <nathanieloshead@gmail.com>