[Bug bootstrap/98128] New: [11 Regression] Bootstrap failed on 32-bit hosts

[Bug bootstrap/98128] New: [11 Regression] Bootstrap failed on 32-bit hosts

[hjl.tools at gmail dot com]

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

            Bug ID: 98128
           Summary: [11 Regression] Bootstrap failed on 32-bit hosts
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: jakub at redhat dot com, jason at redhat dot com
  Target Milestone: ---

On Linux/x86, r11-5702 failed to bootstrap:

../../src-master/gcc/cp/parser.c: In function ??void
cp_parser_late_parsing_default_args(cp_parser*, tree)??:
../../src-master/gcc/cp/parser.c:30592:28: error: ambiguous overload for
??operator[]?? (operand types are ??releasing_vec?? and ??int??)
30592 |       tree parmdecl = parms[i];
      |                            ^
../../src-master/gcc/cp/parser.c:30592:28: note: candidate:
??operator[](releasing_vec::vec_t* {aka vec<tree_node*, va_gc>*}, int)??
(built-in)
In file included from ../../src-master/gcc/cp/parser.c:25:
../../src-master/gcc/cp/cp-tree.h:965:9: note: candidate: ??tree_node*&
releasing_vec::operator[](unsigned int) const??
  965 |   tree& operator[] (unsigned i) const { return (*v)[i]; }
      |         ^~~~~~~~
In file included from ../../src-master/gcc/c-family/c-common.h:26,
                 from ../../src-master/gcc/cp/cp-tree.h:40,
                 from ../../src-master/gcc/cp/parser.c:25:
../../src-master/gcc/cp/parser.c:30621:24: error: ambiguous overload for
??operator[]?? (operand types are ??releasing_vec?? and ??int??)
30621 |       DECL_CHAIN (parms[i]) = parm;
      |                        ^
../../src-master/gcc/tree.h:286:26: note: in definition of macro
??CONTAINS_STRUCT_CHECK??
  286 | (contains_struct_check ((T), (STRUCT), __FILE__, __LINE__,
__FUNCTION__))
      |                          ^
../../src-master/gcc/tree.h:2424:27: note: in expansion of macro ??TREE_CHAIN??
 2424 | #define DECL_CHAIN(NODE) (TREE_CHAIN (DECL_MINIMAL_CHECK (NODE)))
      |                           ^~~~~~~~~~
../../src-master/gcc/tree.h:431:33: note: in expansion of macro
??CONTAINS_STRUCT_CHECK??
  431 | #define DECL_MINIMAL_CHECK(T)   CONTAINS_STRUCT_CHECK (T,
TS_DECL_MINIMAL)
      |                                 ^~~~~~~~~~~~~~~~~~~~~
../../src-master/gcc/tree.h:2424:39: note: in expansion of macro
??DECL_MINIMAL_CHECK??
 2424 | #define DECL_CHAIN(NODE) (TREE_CHAIN (DECL_MINIMAL_CHECK (NODE)))
      |                                       ^~~~~~~~~~~~~~~~~~
../../src-master/gcc/cp/parser.c:30621:7: note: in expansion of macro
??DECL_CHAIN??
30621 |       DECL_CHAIN (parms[i]) = parm;
      |       ^~~~~~~~~~
../../src-master/gcc/cp/parser.c:30621:24: note: candidate:
??operator[](releasing_vec::vec_t* {aka vec<tree_node*, va_gc>*}, int)??
(built-in)
30621 |       DECL_CHAIN (parms[i]) = parm;
      |                        ^
../../src-master/gcc/tree.h:286:26: note: in definition of macro
??CONTAINS_STRUCT_CHECK??

r11-5693 is OK.

[Bug bootstrap/98128] [11 Regression] Bootstrap failed on 32-bit hosts

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2020-12-03
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> ---
It is likely caused by r11-5695.

[Bug bootstrap/98128] [11 Regression] Bootstrap failed on 32-bit hosts

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
--- gcc/cp/parser.c.jj  2020-12-03 20:41:57.194174600 +0100
+++ gcc/cp/parser.c     2020-12-03 20:46:28.626125729 +0100
@@ -30589,7 +30589,7 @@ cp_parser_late_parsing_default_args (cp_
       tree copy;
       unsigned ix;

-      tree parmdecl = parms[i];
+      tree parmdecl = (*parms)[i];
       pushdecl (parmdecl);

       if (!default_arg)
@@ -30618,8 +30618,8 @@ cp_parser_late_parsing_default_args (cp_
   parm = NULL_TREE;
   for (int i = parms->length () - 1; i >= 0; --i)
     {
-      DECL_CHAIN (parms[i]) = parm;
-      parm = parms[i];
+      DECL_CHAIN ((*parms)[i]) = parm;
+      parm = (*parms)[i];
     }

   pop_defarg_context ();

seems to work for me.
So does:
--- gcc/cp/cp-tree.h.jj 2020-12-02 22:55:36.328252325 +0100
+++ gcc/cp/cp-tree.h    2020-12-03 20:49:11.746293467 +0100
@@ -963,6 +963,7 @@ public:

   /* Breaks pointer/value consistency for convenience.  */
   tree& operator[] (unsigned i) const { return (*v)[i]; }
+  tree& operator[] (int i) const { return (*v)[i]; }

   ~releasing_vec() { release_tree_vector (v); }
 private:

[Bug bootstrap/98128] [11 Regression] Bootstrap failed on 32-bit hosts

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marxin at gcc dot gnu.org

--- Comment #3 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to H.J. Lu from comment #1)
> It is likely caused by r11-5695.

Yes, it's since this revision. I see that same issue.

[Bug bootstrap/98128] [11 Regression] Bootstrap failed on 32-bit hosts

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu.org
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jason at gcc dot gnu.org

[Bug bootstrap/98128] [11 Regression] Bootstrap failed on 32-bit hosts

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> ---
Fixed in r11-5733 (with the wrong PR in the comment).