[Bug c++/98885] [modules] forward declaration of classes prevent them from being exported at the point of actual declaration

[cvs-commit at gcc dot gnu.org via Gcc-bugs]

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

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

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

commit r14-5808-gd89903ff29473e6e64f032ecee5c72d1584546dc
Author: Nathaniel Shead <nathanielosh...@gmail.com>
Date:   Sun Nov 12 22:49:45 2023 +1100

    c++: check mismatching exports for class tags [PR98885]

    Checks for exporting a declaration that was previously declared as not
    exported is implemented in 'duplicate_decls', but this doesn't handle
    declarations of classes. This patch adds these checks and slightly
    adjusts the associated error messages for clarity.

            PR c++/98885

    gcc/cp/ChangeLog:

            * decl.cc (duplicate_decls): Adjust error message.
            (xref_tag): Adjust error message. Check exporting decl that is
            already declared as non-exporting.

    gcc/testsuite/ChangeLog:

            * g++.dg/modules/export-1.C: Adjust error messages. Remove
            xfails for working case. Add new test case.

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