[Bug tree-optimization/90973] A suspicious code in tree-vect-loop.c

[marxin at gcc dot gnu.org]

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

--- Comment #5 from Martin Liška  ---
Author: marxin
Date: Wed Jun 26 06:44:58 2019
New Revision: 272671

URL: https://gcc.gnu.org/viewcvs?rev=272671=gcc=rev
Log:
Fix one another thinko in tree-vect-loop.c (PR tree-optimization/90973).

2019-06-26  Martin Liska  

PR tree-optimization/90973
* tree-vect-loop.c (vect_get_known_peeling_cost): Use
epilogue_cost_vec instead of prologue_cost_vec for
a epilogue cost.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-vect-loop.c

[Bug tree-optimization/90973] A suspicious code in tree-vect-loop.c

[marxin at gcc dot gnu.org]

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

Martin Liška  changed:

   What|Removed |Added

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

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

[Bug tree-optimization/90973] A suspicious code in tree-vect-loop.c

[marxin at gcc dot gnu.org]

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

--- Comment #3 from Martin Liška  ---
Author: marxin
Date: Tue Jun 25 08:17:39 2019
New Revision: 272638

URL: https://gcc.gnu.org/viewcvs?rev=272638=gcc=rev
Log:
Properly sum costs in tree-vect-loop.c (PR tree-optimization/90973).

2019-06-25  Martin Liska  

PR tree-optimization/90973
* tree-vect-loop.c (vect_get_known_peeling_cost): Sum retval
of prologue and epilogue.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-vect-loop.c

[Bug tree-optimization/90973] A suspicious code in tree-vect-loop.c

[marxin at gcc dot gnu.org]

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

Martin Liška  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |marxin at gcc dot 
gnu.org

--- Comment #2 from Martin Liška  ---
Then it's a nice catch. Yes, I'll send a patch.

[Bug tree-optimization/90973] A suspicious code in tree-vect-loop.c

[rguenth at gcc dot gnu.org]

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

Richard Biener  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #1 from Richard Biener  ---
Yes (though the return value is mostly unused besides dump reporting).  Care to
patch it?  Patch is pre-approved.