[Bug c++/29985] gdbtypes.h:985: internal-error: field: Assertion `idx >= 0 && idx < num_fields ()' failed.

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

https://sourceware.org/bugzilla/show_bug.cgi?id=29985

--- Comment #11 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom Tromey <tromey@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=1775f8b3804744c22611ed0c9fd2df4f5b4d8641

commit 1775f8b3804744c22611ed0c9fd2df4f5b4d8641
Author: Tom Tromey <tromey@adacore.com>
Date:   Wed Jan 11 12:42:40 2023 -0700

    Increase size of main_type::nfields

    main_type::nfields is a 'short', and has been for many years.  PR
    c++/29985 points out that 'short' is too narrow for an enum that
    contains more than 2^15 constants.

    This patch bumps the size of 'nfields'.  To verify that the field
    isn't directly used, it is also renamed.  Note that this does not
    affect the size of main_type on x86-64 Fedora 36.  And, if it does
    have a negative effect somewhere, it's worth considering that types
    could be shrunk more drastically by using subclasses for the different
    codes.

    This is v2 of this patch, which has these changes:

    * I changed nfields to 'unsigned', per Simon's request.  I looked at
      changing all the uses, but this quickly fans out into a very large
      patch.  (One additional tweak was needed, though.)

    * I wrote a test case.  I discovered that GCC cannot compile a large
      enough C test case, so I resorted to using the DWARF assembler.
      This test doesn't reproduce the crash, but it does fail without the
      patch.

    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29985

-- 
You are receiving this mail because:
You are on the CC list for the bug.