[Bug c++/87554] [8 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2020-02-26 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554

Jason Merrill  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #13 from Jason Merrill  ---
Fixed for 8.4.

[Bug c++/87554] [8 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2020-02-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554

--- Comment #12 from CVS Commits  ---
The releases/gcc-8 branch has been updated by Jason Merrill
:

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

commit r8-10074-ga57528b33be33d4428ac62901d04cf39807d624e
Author: Jason Merrill 
Date:   Tue Feb 25 13:37:18 2020 -0500

PR c++/87554 - ICE with extern template and reference member.

The removed code ended up setting DECL_INITIAL to the INIT_EXPR returned by
split_nonconstant_init, which makes no sense.  This code was added back in
1996, so any rationale is long lost.

gcc/cp/ChangeLog
2020-02-25  Jason Merrill  

PR c++/87554 - ICE with extern template and reference member.
* decl.c (cp_finish_decl): Don't set DECL_INITIAL of external vars.

[Bug c++/87554] [8 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2019-04-18 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554

--- Comment #11 from Jan Hubicka  ---
> > The constructor indeed looks broken to me: it should not have naked
> > var_decl. So I am changing component to C++
> 
> I agree that the C++ front end is wrong here, but I also wonder why cgraph is
> looking at the DECL_INITIAL of a DECL_EXTERNAL variable.

Well, at least for constant variables those are useful for contant
folding.  But I suppose we may take care to get rid of decl initials
of non-readonly externals somewhere soon (we do it eventually as part
of unreachable code removal)

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

[Bug c++/87554] [8 Regression] internal compiler error: in record_reference, at cgraphbuild.c:64

2019-04-18 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87554

Jason Merrill  changed:

   What|Removed |Added

Summary|[8/9 Regression] internal   |[8 Regression] internal
   |compiler error: in  |compiler error: in
   |record_reference, at|record_reference, at
   |cgraphbuild.c:64|cgraphbuild.c:64

--- Comment #10 from Jason Merrill  ---
(In reply to Jan Hubicka from comment #8)
> The constructor indeed looks broken to me: it should not have naked
> var_decl. So I am changing component to C++

I agree that the C++ front end is wrong here, but I also wonder why cgraph is
looking at the DECL_INITIAL of a DECL_EXTERNAL variable.