[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[egallager at gcc dot gnu.org]

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

Eric Gallager  changed:

   What|Removed |Added

 Status|WAITING |RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Eric Gallager  ---
(In reply to Eric Gallager from comment #5)
> (In reply to Eric Gallager from comment #4)
> > (In reply to Vladimir Makarov from comment #3)
> > > Author: vmakarov
> > > Date: Fri Sep 29 17:15:24 2017
> > > New Revision: 253299
> > > 
> > > URL: https://gcc.gnu.org/viewcvs?rev=253299=gcc=rev
> > > Log:
> > > 2017-09-29  Vladimir Makarov  
> > > 
> > >   PR rtl-optimization/82338
> > >   * lra-constraints.c (inherit_in_ebb): Check usage_insns check.
> > > 
> > > 
> > > Modified:
> > > trunk/gcc/ChangeLog
> > > trunk/gcc/lra-constraints.c
> > 
> > Did this fix it?
> 
> WAITING on a reply

Assuming this has been fixed.

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[egallager at gcc dot gnu.org]

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

Eric Gallager  changed:

   What|Removed |Added

 Status|NEW |WAITING

--- Comment #5 from Eric Gallager  ---
(In reply to Eric Gallager from comment #4)
> (In reply to Vladimir Makarov from comment #3)
> > Author: vmakarov
> > Date: Fri Sep 29 17:15:24 2017
> > New Revision: 253299
> > 
> > URL: https://gcc.gnu.org/viewcvs?rev=253299=gcc=rev
> > Log:
> > 2017-09-29  Vladimir Makarov  
> > 
> > PR rtl-optimization/82338
> > * lra-constraints.c (inherit_in_ebb): Check usage_insns check.
> > 
> > 
> > Modified:
> > trunk/gcc/ChangeLog
> > trunk/gcc/lra-constraints.c
> 
> Did this fix it?

WAITING on a reply

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[egallager at gcc dot gnu.org]

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

Eric Gallager  changed:

   What|Removed |Added

 CC||egallager at gcc dot gnu.org

--- Comment #4 from Eric Gallager  ---
(In reply to Vladimir Makarov from comment #3)
> Author: vmakarov
> Date: Fri Sep 29 17:15:24 2017
> New Revision: 253299
> 
> URL: https://gcc.gnu.org/viewcvs?rev=253299=gcc=rev
> Log:
> 2017-09-29  Vladimir Makarov  
> 
>   PR rtl-optimization/82338
>   * lra-constraints.c (inherit_in_ebb): Check usage_insns check.
> 
> 
> Modified:
> trunk/gcc/ChangeLog
> trunk/gcc/lra-constraints.c

Did this fix it?

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[vmakarov at gcc dot gnu.org]

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

--- Comment #3 from Vladimir Makarov  ---
Author: vmakarov
Date: Fri Sep 29 17:15:24 2017
New Revision: 253299

URL: https://gcc.gnu.org/viewcvs?rev=253299=gcc=rev
Log:
2017-09-29  Vladimir Makarov  

PR rtl-optimization/82338
* lra-constraints.c (inherit_in_ebb): Check usage_insns check.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/lra-constraints.c

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[marxin at gcc dot gnu.org]

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

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-09-29
 CC||marxin at gcc dot gnu.org
 Ever confirmed|0   |1

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[vmakarov at gcc dot gnu.org]

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

--- Comment #2 from Vladimir Makarov  ---
  Thank you for reporting this, especially for reducing the test case.  The bug
is not dangerous, it does not result in wrong code generation but it might
result in worse code.

  I reproduced it.  It seams the validity (obsoleteness) of data was not
checked first.  I'll make a patch and commit it tomorrow.  If I am not able to
make it on this week, I'll do it in a week.

[Bug rtl-optimization/82338] valgrind error in inherit_in_ebb

[dcb314 at hotmail dot com]

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

--- Comment #1 from David Binderman  ---
12 hours reducing leads to this C++ code:

extern "C" {
void a();
void *memset(void *, int, unsigned long);
}
struct b {
  int c;
  int d;
} e[5000], *f;
int g;
int h;
int i;
int j, k;
void l(int);
int m;
int o;
void load() {
  int n;
  memset(e, 0, sizeof(e));
  for (; k;)
;
  a();
  for (; m < o; m++)
for (; i; n++) {
  e[j].c = g;
  if (f[h].d && n == i)
l(-1);
}
}