[Bug c/89946] [8 Regression] ICE in assemble_start_function, at varasm.c:1871

2019-05-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89946

Jakub Jelinek  changed:

   What|Removed |Added

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

--- Comment #6 from Jakub Jelinek  ---
Fixed for 8.4+ too.

[Bug c/89946] [8 Regression] ICE in assemble_start_function, at varasm.c:1871

2019-04-30 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89946

--- Comment #5 from Jakub Jelinek  ---
Author: jakub
Date: Tue Apr 30 21:03:11 2019
New Revision: 270749

URL: https://gcc.gnu.org/viewcvs?rev=270749=gcc=rev
Log:
Backported from mainline
2019-04-12  Jakub Jelinek  

PR c/89946
* varasm.c (assemble_start_function): Don't use tree_fits_uhwi_p
and gcc_unreachable if it fails, just call tree_to_uhwi which
verifies that too.  Test TREE_CHAIN instead of list_length > 1.
Start warning message with a lower-case letter.  Formatting fixes.

* c-attribs.c (handle_patchable_function_entry_attribute): Add
function comment.  Warn if arguments of the attribute are not positive
integer constants.

* c-c++-common/pr89946.c: New test.

Added:
branches/gcc-8-branch/gcc/testsuite/c-c++-common/pr89946.c
Modified:
branches/gcc-8-branch/gcc/ChangeLog
branches/gcc-8-branch/gcc/c-family/ChangeLog
branches/gcc-8-branch/gcc/c-family/c-attribs.c
branches/gcc-8-branch/gcc/testsuite/ChangeLog
branches/gcc-8-branch/gcc/varasm.c

[Bug c/89946] [8 Regression] ICE in assemble_start_function, at varasm.c:1871

2019-04-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89946

Jakub Jelinek  changed:

   What|Removed |Added

Summary|[8/9 Regression] ICE in |[8 Regression] ICE in
   |assemble_start_function, at |assemble_start_function, at
   |varasm.c:1871   |varasm.c:1871

--- Comment #4 from Jakub Jelinek  ---
Fixed on the trunk so far.