[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2017-06-22 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

Martin Liška  changed:

   What|Removed |Added

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

--- Comment #8 from Martin Liška  ---
Fixed on GCC 6.x, no plan to backport to GCC 5.x.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2017-06-22 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

--- Comment #7 from Martin Liška  ---
Author: marxin
Date: Thu Jun 22 11:14:13 2017
New Revision: 249511

URL: https://gcc.gnu.org/viewcvs?rev=249511=gcc=rev
Log:
Backport r241380

2017-06-22  Martin Liska  

Backport from mainline
2016-10-20  Martin Liska  

PR lto/78049
* lto-streamer-in.c (fixup_call_stmt_edges_1): Replace value
comparison with STMT_UID_NOT_IN_RANGE.
(fixup_call_stmt_edges): Do not fixup edges of a thunk in
LTRANS.

Modified:
branches/gcc-6-branch/gcc/ChangeLog
branches/gcc-6-branch/gcc/lto-streamer-in.c

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2017-05-31 Thread pinskia at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|7.0 |5.5

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2017-05-30 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

Martin Liška  changed:

   What|Removed |Added

 Status|RESOLVED|ASSIGNED
   Last reconfirmed||2017-05-30
 Resolution|FIXED   |---
 Ever confirmed|0   |1

--- Comment #6 from Martin Liška  ---
I'm planning to backport the patch to both active branches.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Martin Liška  ---
Fixed.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

--- Comment #4 from Martin Liška  ---
Author: marxin
Date: Thu Oct 20 15:30:17 2016
New Revision: 241380

URL: https://gcc.gnu.org/viewcvs?rev=241380=gcc=rev
Log:
Do not fixup edges for a thunk in LTRANS (PR lto/78049)

PR lto/78049
* lto-streamer-in.c (fixup_call_stmt_edges_1): Replace value
comparison with STMT_UID_NOT_IN_RANGE.
(fixup_call_stmt_edges): Do not fixup edges of a thunk in
LTRANS.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/lto-streamer-in.c

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

--- Comment #3 from Martin Liška  ---
Having a patch that I'll send shortly to ML.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

--- Comment #2 from Martin Liška  ---
Hm, cedge->lto_stmt_uid == 0, thus adjusting fatal_error guard condition and
I'm seeking why it has the wrong value.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

--- Comment #1 from Richard Biener  ---
static void
fixup_call_stmt_edges_1 (struct cgraph_node *node, gimple **stmts,
 struct function *fn)
{
  struct cgraph_edge *cedge;
  struct ipa_ref *ref = NULL;
  unsigned int i;

  for (cedge = node->callees; cedge; cedge = cedge->next_callee)
{
  if (gimple_stmt_max_uid (fn) < cedge->lto_stmt_uid)
fatal_error (input_location,
 "Cgraph edge statement index out of range");
901cedge->call_stmt = as_a  (stmts[cedge->lto_stmt_uid - 1]);
  if (!cedge->call_stmt)

so it looks like cedge->lto_stmt_uid is out of bounds or out-of-sync.

[Bug lto/78049] [7 Regression] ICE in gcc/lto-streamer-in.c:901 when building Firefox with LTO and -O3

2016-10-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78049

Martin Liška  changed:

   What|Removed |Added

   Keywords||ice-on-valid-code
 CC||hubicka at gcc dot gnu.org,
   ||octoploid at yandex dot com,
   ||rguenth at gcc dot gnu.org
   Target Milestone|--- |7.0