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

--- Comment #2 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The trunk branch has been updated by Marek Polacek <mpola...@gcc.gnu.org>:

https://gcc.gnu.org/g:725c68c54c265fe7f6fc7babff7139f3161bdfa6

commit r14-6024-g725c68c54c265fe7f6fc7babff7139f3161bdfa6
Author: Marek Polacek <pola...@redhat.com>
Date:   Tue Nov 28 14:44:24 2023 -0500

    c++: wrong ambiguity in accessing static field [PR112744]

    Given

      struct A { constexpr static int a = 0; };
      struct B : A {};
      struct C : A {};
      struct D : B, C {};

    we give the "'A' is an ambiguous base of 'D'" error for

      D{}.A::a;

    which seems wrong: 'a' is a static data member so there is only one copy
    so it can be unambiguously referred to even if there are multiple A
    objects.  clang++/MSVC/icx agree.

    This patch uses ba_any: [class.access.base] requires conversion to a unique
    base subobject for non-static data members, but it does not require that
the
    base be unique or accessible for static data members.

            PR c++/112744

    gcc/cp/ChangeLog:

            * typeck.cc (finish_class_member_access_expr): When accessing
            a static data member, use ba_any for lookup_base.

    gcc/testsuite/ChangeLog:

            * g++.dg/lookup/scoped11.C: New test.
            * g++.dg/lookup/scoped12.C: New test.
            * g++.dg/lookup/scoped13.C: New test.
            * g++.dg/lookup/scoped14.C: New test.
            * g++.dg/lookup/scoped15.C: New test.
  • [Bug c++/112744] Nested name sp... cvs-commit at gcc dot gnu.org via Gcc-bugs

Reply via email to