[Bug c++/113580] New: -Wunused-parameter in template imported from module causes segmentation fault

[Bug c++/113580] New: -Wunused-parameter in template imported from module causes segmentation fault

[michael.kenzel at gmail dot com]

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

            Bug ID: 113580
           Summary: -Wunused-parameter in template imported from module
                    causes segmentation fault
           Product: gcc
           Version: 13.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: michael.kenzel at gmail dot com
  Target Milestone: ---

The following example will reproduce the issue:

        // A.ixx
        export module A;

        export void fun(auto&& x) {}

        // main.cpp
        import A;

        int main()
        {
                fun(42);
        }


g++ -std=c++23 -fmodules-ts -c -x c++ A.ixx
g++ -std=c++23 -Wunused-parameter -fmodules-ts main.cpp A.o 

results in

Segmentation fault
0x7ff37686550f ???
        ./signal/../sysdeps/unix/sysv/linux/x86_64/libc_sigaction.c:0
0x7ff37697e159 __strlen_avx2
        ../sysdeps/x86_64/multiarch/strlen-avx2.S:76
0x7ff3768506c9 __libc_start_call_main
        ../sysdeps/nptl/libc_start_call_main.h:58
0x7ff376850784 __libc_start_main_impl
        ../csu/libc-start.c:360

[Bug c++/113580] -Wunused-parameter in template imported from module causes segmentation fault

[cvs-commit at gcc dot gnu.org]

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

--- 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:ec57d183d35412aa5e0bcd7a448ccb75a4e1eab7

commit r14-8462-gec57d183d35412aa5e0bcd7a448ccb75a4e1eab7
Author: Nathaniel Shead <nathanieloshead@gmail.com>
Date:   Fri Jan 26 16:55:52 2024 +1100

    c++: Stream additional fields for DECL_STRUCT_FUNCTION [PR113580]

    Currently the DECL_STRUCT_FUNCTION for a declaration is always
    reconstructed from scratch. This causes issues though, as some fields
    used by other parts of the compiler (in this case, specifically
    'function_{start,end}_locus') are then not correctly initialised. This
    patch makes sure that these fields are also read and written.

            PR c++/113580

    gcc/cp/ChangeLog:

            * module.cc (struct post_process_data): Create.
            (trees_in::post_decls): Use.
            (trees_in::post_process): Return entire vector at once.
            Change overload to take post_process_data instead of tree.
            (trees_out::write_function_def): Write needed flags from
            DECL_STRUCT_FUNCTION.
            (trees_in::read_function_def): Read them and pass to
            post_process.
            (module_state::read_cluster): Write flags into cfun.

    gcc/testsuite/ChangeLog:

            * g++.dg/modules/pr113580_a.C: New test.
            * g++.dg/modules/pr113580_b.C: New test.

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

[Bug c++/113580] -Wunused-parameter in template imported from module causes segmentation fault

[nshead at gcc dot gnu.org]

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

Nathaniel Shead <nshead at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
             Blocks|                            |103524
                 CC|                            |nshead at gcc dot gnu.org
   Target Milestone|---                         |14.0
           Assignee|unassigned at gcc dot gnu.org      |nshead at gcc dot gnu.org
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Nathaniel Shead <nshead at gcc dot gnu.org> ---
Fixed for GCC 14.


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103524
[Bug 103524] [meta-bug] modules issue

[Bug c++/113580] -Wunused-parameter in template imported from module causes segmentation fault

[ppalka at gcc dot gnu.org]

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

Patrick Palka <ppalka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |huscar at gmail dot com

--- Comment #3 from Patrick Palka <ppalka at gcc dot gnu.org> ---
*** Bug 106951 has been marked as a duplicate of this bug. ***