[Bug middle-end/90796] [8/9 Regression] GCC: O2 vs O3 output differs on simple test

2019-11-20 Thread matz at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90796

--- Comment #17 from Michael Matz  ---
Author: matz
Date: Wed Nov 20 16:51:10 2019
New Revision: 278512

URL: https://gcc.gnu.org/viewcvs?rev=278512=gcc=rev
Log:
Fix PR90796

   PR middle-end/90796
   * gimple-loop-jam.c (any_access_function_variant_p): New function.
   (adjust_unroll_factor): Use it to constrain safety, new parameter.
   (tree_loop_unroll_and_jam): Adjust call and profitable unroll factor.

testsuite/
   Backport from mainline
   PR middle-end/90796
   * gcc.dg/unroll-and-jam.c: Disable loop-invariant motion and adjust.

   PR middle-end/90796
   * gcc.dg/unroll-and-jam.c: Add three invalid and one valid case.

Modified:
branches/gcc-9-branch/gcc/ChangeLog
branches/gcc-9-branch/gcc/gimple-loop-jam.c
branches/gcc-9-branch/gcc/testsuite/ChangeLog
branches/gcc-9-branch/gcc/testsuite/gcc.dg/unroll-and-jam.c

[Bug middle-end/90796] [8/9 Regression] GCC: O2 vs O3 output differs on simple test

2019-11-20 Thread matz at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90796

--- Comment #16 from Michael Matz  ---
(In reply to Jakub Jelinek from comment #14)
> Time to backport now?

Hmpf, I've actually done the regstrapping for gcc9 already but then forgot to
submit.  Thanks for the reminder.

[Bug middle-end/90796] [8/9 Regression] GCC: O2 vs O3 output differs on simple test

2019-11-20 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90796

Jakub Jelinek  changed:

   What|Removed |Added

 CC||vsevolod.livinskij at frtk dot 
ru

--- Comment #15 from Jakub Jelinek  ---
*** Bug 91240 has been marked as a duplicate of this bug. ***

[Bug middle-end/90796] [8/9 Regression] GCC: O2 vs O3 output differs on simple test

2019-11-20 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90796

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #14 from Jakub Jelinek  ---
Time to backport now?

[Bug middle-end/90796] [8/9 Regression] GCC: O2 vs O3 output differs on simple test

2019-10-22 Thread matz at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90796

Michael Matz  changed:

   What|Removed |Added

Summary|[8/9/10 Regression] GCC: O2 |[8/9 Regression] GCC: O2 vs
   |vs O3 output differs on |O3 output differs on simple
   |simple test |test
  Known to fail|10.0|

--- Comment #13 from Michael Matz  ---
Fixed in trunk so far.  Will be backporting in a few days.