[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

Richard Biener  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
  Known to work||7.1.0
 Resolution|--- |FIXED

--- Comment #11 from Richard Biener  ---
I would say so.  Fixed in GCC 7.

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #10 from Martin Liška  ---
What's the status of the PR? Can we close it?

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[cltang at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #9 from Chung-Lin Tang  ---
Author: cltang
Date: Thu Sep 15 10:00:14 2016
New Revision: 240156

URL: https://gcc.gnu.org/viewcvs?rev=240156&root=gcc&view=rev
Log:
2016-09-15  Chung-Lin Tang  

Backport from mainline
2016-09-15  Chung-Lin Tang  

PR fortran/72743
* ipa-icf.c (set_alias_uids): New function.
(sem_variable::merge): Use set_alias_uids to set DECL_PT_UID of
all the merged variable's referring aliases.

testsuite/
* gfortran.dg/goacc/pr72743.f90: New test.


Added:
branches/gomp-4_0-branch/gcc/testsuite/gfortran.dg/goacc/pr72743.f90
Modified:
branches/gomp-4_0-branch/gcc/ChangeLog.gomp
branches/gomp-4_0-branch/gcc/ipa-icf.c
branches/gomp-4_0-branch/gcc/testsuite/ChangeLog.gomp

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[cltang at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #8 from Chung-Lin Tang  ---
Author: cltang
Date: Thu Sep 15 09:46:36 2016
New Revision: 240155

URL: https://gcc.gnu.org/viewcvs?rev=240155&root=gcc&view=rev
Log:
2016-09-15  Chung-Lin Tang  

PR fortran/72743
* ipa-icf.c (set_alias_uids): New function.
(sem_variable::merge): Use set_alias_uids to set DECL_PT_UID of
all the merged variable's referring aliases.

testsuite/
* gfortran.dg/goacc/pr72743.f90: New test.


Added:
trunk/gcc/testsuite/gfortran.dg/goacc/pr72743.f90
Modified:
trunk/gcc/ChangeLog
trunk/gcc/ipa-icf.c
trunk/gcc/testsuite/ChangeLog

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[tschwinge at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

Thomas Schwinge  changed:

   What|Removed |Added

   Keywords||openacc
 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |cltang at gcc dot 
gnu.org

--- Comment #7 from Thomas Schwinge  ---
Patch review in .

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[burnus at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

Tobias Burnus  changed:

   What|Removed |Added

 CC||burnus at gcc dot gnu.org

--- Comment #6 from Tobias Burnus  ---
Posted patch: https://gcc.gnu.org/ml/gcc-patches/2016-08/msg02114.html

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[rguenther at suse dot de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #5 from rguenther at suse dot de  ---
On Tue, 9 Aug 2016, tschwinge at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743
> 
> Thomas Schwinge  changed:
> 
>What|Removed |Added
> 
>Last reconfirmed|2016-07-29 00:00:00 |2016-8-9
>  CC||marxin at gcc dot gnu.org,
>||rguenth at gcc dot gnu.org,
>||tschwinge at gcc dot gnu.org
> 
> --- Comment #4 from Thomas Schwinge  ---
> Confirmed.  Probably Dominique's x86_64-apple-darwin15 configuration has
> something differently in symbol visibility or some such, which hides this
> problem.
> 
> Richard, you're probably thinking of PR70856 when you say this sounds 
> familiar.
>  Indeed this looks similar, but it's still a new/different problem.  Do you or
> Martin (who fixed PR70856) have any plans to look into this issue here?

Low on my priority list.  Who is creating the clone this time?

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[tschwinge at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

Thomas Schwinge  changed:

   What|Removed |Added

   Last reconfirmed|2016-07-29 00:00:00 |2016-8-9
 CC||marxin at gcc dot gnu.org,
   ||rguenth at gcc dot gnu.org,
   ||tschwinge at gcc dot gnu.org

--- Comment #4 from Thomas Schwinge  ---
Confirmed.  Probably Dominique's x86_64-apple-darwin15 configuration has
something differently in symbol visibility or some such, which hides this
problem.

Richard, you're probably thinking of PR70856 when you say this sounds familiar.
 Indeed this looks similar, but it's still a new/different problem.  Do you or
Martin (who fixed PR70856) have any plans to look into this issue here?

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #3 from Dominique d'Humieres  ---
> WORKSFORME on x86_64-apple-darwin15 for all the revisions (from 4.8
> up to trunk: 7.0) and options I have tried.

with/without -fopenacc for gcc-5 up to trunk.

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[dominiq at lps dot ens.fr]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

--- Comment #2 from Dominique d'Humieres  ---
WORKSFORME on x86_64-apple-darwin15 for all the revisions (from 4.8 up to
trunk: 7.0) and options I have tried.

[Bug fortran/72743] ICE in get_constraint_for_ssa_var, at tree-ssa-structalias.c:2958

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72743

Richard Biener  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2016-07-29
 Ever confirmed|0   |1
  Known to fail||6.1.1, 7.0

--- Comment #1 from Richard Biener  ---
Hmm, this sounds familiar (there is/was a dup I think):

#1  0x01188c25 in get_constraint_for_ssa_var (t=
Python Exception  There is no member or method named
m_vecpfx.: 
, results=0x7fffd5f0, address_p=true)
at /space/rguenther/src/svn/trunk/gcc/tree-ssa-structalias.c:2942
2942  gcc_assert ((! DECL_PT_UID_SET_P (node->decl)
(gdb) l
2937{
2938  node = node->ultimate_alias_target ();
2939  /* Canonicalize the PT uid of all aliases to the ultimate
target.
2940 ???  Hopefully the set of aliases can't change in a way
that
2941 changes the ultimate alias target.  */
2942  gcc_assert ((! DECL_PT_UID_SET_P (node->decl)
2943   || DECL_PT_UID (node->decl) == DECL_UID
(node->decl))
2944  && (! DECL_PT_UID_SET_P (t)
2945  || DECL_PT_UID (t) == DECL_UID
(node->decl)));
2946  DECL_PT_UID (t) = DECL_UID (node->decl);