[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-11 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Jakub Jelinek  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Assignee|unassigned at gcc dot gnu.org  |jakub at gcc dot gnu.org

--- Comment #12 from Jakub Jelinek  ---
Should be fixed now.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-11 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #11 from Jakub Jelinek  ---
Author: jakub
Date: Tue Dec 11 20:37:53 2018
New Revision: 267032

URL: https://gcc.gnu.org/viewcvs?rev=267032&root=gcc&view=rev
Log:
PR c++/87861
* class.c (build_vtbl_initializer): For TARGET_VTABLE_USES_DESCRIPTORS
bump index for each added word.
* constexpr.c (find_array_ctor_elt): Add forward declaration.
(cxx_eval_call_expression): Handle TARGET_VTABLE_USES_DESCRIPTORS
vtable calls.
(cxx_eval_constant_expression) : Divide token
by TARGET_VTABLE_USES_DESCRIPTORS if non-zero.

Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/class.c
trunk/gcc/cp/constexpr.c

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-08 Thread sch...@linux-m68k.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #10 from Andreas Schwab  ---
http://gcc.gnu.org/ml/gcc-testresults/2018-12/msg00950.html

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #9 from Jeffrey A. Law  ---
Jakub -- with your patch and qsort checking hacked off I got a successful ia64
bootstrap.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #8 from Jeffrey A. Law  ---
We certainly get further with your patch -- we fail qsort checking during the
stage2 build, but that's nothing new.  ia64 has run afoul of the qsort checking
since the day qsort checking was introduced.  I'll hack around it and let the
bootstrap continue.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #7 from Jeffrey A. Law  ---
I've still got my ia64 beaker box from yesterday provisioned.  I'll spin your
patch.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Jakub Jelinek  changed:

   What|Removed |Added

  Attachment #45178|0   |1
is obsolete||

--- Comment #6 from Jakub Jelinek  ---
Created attachment 45182
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45182&action=edit
gcc9-pr87861.patch

Updated patch, so that it doesn't fail to compile due to warning on non-ia64.

Andreas, could you please bootstrap/regtest this?

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #5 from Jakub Jelinek  ---
Created attachment 45178
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45178&action=edit
gcc9-ia64-constexpr-virtual.patch

Here is untested patch to fix all g++.dg/cpp2a/constexpr-virtual*.C failures,
not sure if it is related or not.
I don't have access to ia64 so can't really test this.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #4 from Jakub Jelinek  ---
Could you please attach preprocessed source on which you get this ICE?
Thanks.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-12-06 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Jeffrey A. Law  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2018-12-06
 CC||law at redhat dot com
 Ever confirmed|0   |1

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-11-02 Thread sch...@linux-m68k.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #3 from Andreas Schwab  ---
http://gcc.gnu.org/ml/gcc-patches/2018-09/msg01052.html

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-11-02 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

--- Comment #2 from Marek Polacek  ---
It's unlikely that this is caused by my latest patch implementing P0846R0, that
only triggers with C++2a.

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-11-02 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Marek Polacek  changed:

   What|Removed |Added

 CC||mpolacek at gcc dot gnu.org

--- Comment #1 from Marek Polacek  ---
When did this start?

[Bug c++/87861] [9 regression] ICE in output_constructor_regular_field, at varasm.c:5165

2018-11-02 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87861

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|--- |9.0