[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-26 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #2 from Jakub Jelinek jakub at gcc dot gnu.org ---
Created attachment 34575
  -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34575action=edit
gcc5-pr64730.patch

I don't think there was anything wrong with the patch, just there is a problem
that a non-NULL edge-call_stmt necessarily doesn't mean the call has a known
location.

[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-26 Thread pinskia at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

Andrew Pinski pinskia at gcc dot gnu.org changed:

   What|Removed |Added

   Assignee|pinskia at gcc dot gnu.org |jakub at gcc dot gnu.org

--- Comment #3 from Andrew Pinski pinskia at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #2)
 Created attachment 34575 [details]
 gcc5-pr64730.patch
 
 I don't think there was anything wrong with the patch, just there is a
 problem that a non-NULL edge-call_stmt necessarily doesn't mean the call
 has a known location.

Totally agree.  Assigning to Jakub since he has a patch.

[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-26 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

--- Comment #4 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Mon Jan 26 21:33:11 2015
New Revision: 220139

URL: https://gcc.gnu.org/viewcvs?rev=220139root=gccview=rev
Log:
PR ipa/64730
* ipa-inline.c (inline_small_functions): Print unknown even
if edge-call_stmt is non-NULL, but has builtins or unknown
location.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/ipa-inline.c
trunk/gcc/testsuite/ChangeLog

[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-26 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

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

--- Comment #5 from Jakub Jelinek jakub at gcc dot gnu.org ---
Should be fixed now.

[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-22 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Priority|P3  |P1
Summary|g++.dg/ipa/pr64049-1.C ICE: |[5 Regression]
   |SEGV when printing NULL |g++.dg/ipa/pr64049-1.C ICE:
   ||SEGV when printing NULL

[Bug ipa/64730] [5 Regression] g++.dg/ipa/pr64049-1.C ICE: SEGV when printing NULL

2015-01-22 Thread pinskia at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64730

Andrew Pinski pinskia at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2015-01-22
   Assignee|unassigned at gcc dot gnu.org  |pinskia at gcc dot 
gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Andrew Pinski pinskia at gcc dot gnu.org ---
I will take this over.  I think Naveen swapped one expression that should not
have been swapped around.