[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

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

--- Comment #10 from Jakub Jelinek  ---
Author: jakub
Date: Thu Oct 11 18:35:52 2018
New Revision: 265053

URL: https://gcc.gnu.org/viewcvs?rev=265053=gcc=rev
Log:
PR c++/85070
* g++.dg/cpp0x/pr85070.C: Change effective target for diagnostics from
c++14_only to c++14.

Modified:
branches/gcc-8-branch/gcc/testsuite/ChangeLog
branches/gcc-8-branch/gcc/testsuite/g++.dg/cpp0x/pr85070.C

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

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

--- Comment #9 from Jakub Jelinek  ---
Author: jakub
Date: Thu Oct 11 18:35:11 2018
New Revision: 265052

URL: https://gcc.gnu.org/viewcvs?rev=265052=gcc=rev
Log:
PR c++/85070
* g++.dg/cpp0x/pr85070.C: Change effective target for diagnostics from
c++14_only to c++14.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/cpp0x/pr85070.C

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-09-24 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

Paolo Carlini  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Assignee|nathan at gcc dot gnu.org  |unassigned at gcc dot 
gnu.org

--- Comment #8 from Paolo Carlini  ---
Fixed for 8.3.

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-09-24 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

--- Comment #7 from paolo at gcc dot gnu.org  ---
Author: paolo
Date: Mon Sep 24 17:44:02 2018
New Revision: 264542

URL: https://gcc.gnu.org/viewcvs?rev=264542=gcc=rev
Log:
/cp
2018-09-24  Paolo Carlini  

PR c++/85070
* method.c (lazily_declare_fn): During error-recovery add_method
may return false.

/testsuite
2018-09-24  Paolo Carlini  

PR c++/85070
* g++.dg/cpp0x/pr85070.C: New.

Added:
branches/gcc-8-branch/gcc/testsuite/g++.dg/cpp0x/pr85070.C
Modified:
branches/gcc-8-branch/gcc/cp/ChangeLog
branches/gcc-8-branch/gcc/cp/method.c
branches/gcc-8-branch/gcc/testsuite/ChangeLog

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-09-24 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

--- Comment #6 from paolo at gcc dot gnu.org  ---
Author: paolo
Date: Mon Sep 24 17:42:45 2018
New Revision: 264541

URL: https://gcc.gnu.org/viewcvs?rev=264541=gcc=rev
Log:
/cp
2018-09-24  Paolo Carlini  

PR c++/85070
* method.c (lazily_declare_fn): During error-recovery add_method
may return false.

/testsuite
2018-09-24  Paolo Carlini  

PR c++/85070
* g++.dg/cpp0x/pr85070.C: New.

Added:
trunk/gcc/testsuite/g++.dg/cpp0x/pr85070.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/method.c
trunk/gcc/testsuite/ChangeLog

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-09-20 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

--- Comment #5 from Nathan Sidwell  ---
|| errorcount sounds completely plausible

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-09-20 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

Paolo Carlini  changed:

   What|Removed |Added

 CC||paolo.carlini at oracle dot com

--- Comment #4 from Paolo Carlini  ---
Nathan, I stumbled into this minor regression and noticed that the ICE happens
exactly at the gcc_assert in lazily_declare_fn that you added as part of
r248285:

  /* Add it to the class  */
  bool added = add_method (type, fn, false);
  gcc_assert (added);

but, for the testcase, during error-recovery added is false because add_method
issued a correct - I believe - error_at + inform. Thus, is it only matter of
loosening a bit the gcc_assert, eg added || errorcount, or something deeper is
going on?

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

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

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|8.2 |8.3

--- Comment #3 from Jakub Jelinek  ---
GCC 8.2 has been released.

[Bug c++/85070] [8/9 Regression] ICE on C++ code: in lazily_declare_fn, at cp/method.c:2409

2018-05-02 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85070

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|8.0 |8.2

--- Comment #2 from Jakub Jelinek  ---
GCC 8.1 has been released.