[Bug tree-optimization/43065] [4.5 Regression] gcc.c-torture/execute/20051215-1.c is miscompiled with -fgraphite-identity

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #12 from spop at gcc dot gnu dot org 2010-03-08 17:53 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug middle-end/42130] [graphite-branch] dealII fails

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #11 from spop at gcc dot gnu dot org 2010-03-08 17:54 --- Fixed by http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157287 -- spop at gcc dot gnu dot org changed: What|Removed

[Bug tree-optimization/42644] [4.5 Regression][graphite] wrong code for -fgraphite-identify in 183.equake

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-08 17:55 --- Fixed by http://gcc.gnu.org/viewcvs?view=revision&revision=157286 -- spop at gcc dot gnu dot org changed: What|Removed |A

[Bug middle-end/43274] [4.5 regression] Revision 157244 failed libgomp.graphite/force-parallel-2.c

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-08 22:49 --- Hi, is this fail on i686-linux? On x86_64-unknown-linux-gnu I see no fails in libgomp/ for graphite.exp . Sebastian -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43274

[Bug middle-end/41334] gcc.dg/graphite/run-id-1.c fails execution test

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-08 22:59 --- Hi Jack, is gcc.dg/graphite/run-id-1.c still failing on x86_64-apple-darwin10 or has it been fixed since then? Thanks, Sebastian -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41334

[Bug tree-optimization/42729] [4.5 Regression] "-fcompare-debug failure" with "-O1 -fgraphite-identity"

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-08 23:09 --- Ok, then let's close this PR. -- spop at gcc dot gnu dot org changed: What|Removed |

[Bug middle-end/41334] gcc.dg/graphite/run-id-1.c fails execution test

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-08 23:10 --- Ok, then let's close this PR. -- spop at gcc dot gnu dot org changed: What|Removed |

[Bug tree-optimization/42729] [4.5 Regression] "-fcompare-debug failure" with "-O1 -fgraphite-identity"

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-08 23:10 --- Sorry, I didn't meant this. -- spop at gcc dot gnu dot org changed: What|Removed |

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #15 from spop at gcc dot gnu dot org 2010-03-08 23:16 --- Yes. I think it is important to understand what is miscompiled with the graphite identity. I will try to reduce this. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42181

[Bug middle-end/43300] [4.5 Regression] ICE: in emit_move_insn, at expr.c:3432

2010-03-08 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-09 00:49 --- Ok, so then this is not related to Graphite. I'm removing the Graphite flag from the subject. -- spop at gcc dot gnu dot org changed: What|Removed |

[Bug tree-optimization/43306] [4.5 Regression] ICE: in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:713 with -O1 -fstrict-overflow -fgraphite-identity

2010-03-09 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-09 16:48 --- Mine. -- spop at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at

[Bug tree-optimization/43306] [4.5 Regression] ICE: in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:713 with -O1 -fstrict-overflow -fgraphite-identity

2010-03-09 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-09 19:39 --- Subject: Bug 43306 Author: spop Date: Tue Mar 9 19:39:27 2010 New Revision: 157322 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157322 Log: Fix PR43306: correct evolution_function_right_is_int

[Bug tree-optimization/43306] [4.5 Regression] ICE: in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:713 with -O1 -fstrict-overflow -fgraphite-identity

2010-03-09 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-09 21:07 --- This is different than PR42181. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43306

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-09 Thread spop at gcc dot gnu dot org
--- Comment #23 from spop at gcc dot gnu dot org 2010-03-09 22:31 --- Thanks for reducing this testcase. On the further reduced kernel: SUBROUTINE SPECTOP(Dr,N) IMPLICIT REAL*8(A-H,o-Z) DIMENSION d1(0:32,0:32) , Dr(0:32,0:32) , x(0:32) REAL*8 Dr DO k = 1

[Bug middle-end/42339] [graphite] Autopar on SPEC 2006 fails

2010-03-11 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-11 13:56 --- Fixed as shown by the automatic tester results today: http://groups.google.com/group/gcc-graphite-test/browse_thread/thread/cd55ed193db3862a# -- spop at gcc dot gnu dot org changed: What|Removed

[Bug tree-optimization/43349] [graphite] very long compile time for gamess with -floop-interchange

2010-03-12 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-12 22:57 --- It looks like 10 is still a too high number for PARAM_GRAPHITE_MAX_NB_SCOP_PARAMS Could you try with --param graphite-max-nb-scop-params=5 I think we should improve the parameter detection to improve the compile time

[Bug tree-optimization/43349] [graphite] very long compile time for gamess with -floop-interchange

2010-03-12 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-12 23:01 --- The exponential behavior of PPL has been discussed here: http://groups.google.com/group/gcc-graphite/browse_thread/thread/db215cefd9bb9c8d -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43349

[Bug tree-optimization/43354] [4.5 Regression] function �pofun2' from xplor-nih ICEs compiler at -O2 -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-13 17:13 --- Reduced to: SUBROUTINE POFUN2(DIM,GRDENT,FPART,FPARTL) DOUBLE PRECISION GRDENT(*) DOUBLE COMPLEX FPART(*) DOUBLE COMPLEX FPARTL(*) INTEGER REFLCT IF (DIM.GT.1) THEN ABCS3

[Bug tree-optimization/43349] [graphite] very long compile time for gamess with -floop-interchange

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-03-13 17:27 --- Subject: Bug 43349 Author: spop Date: Sat Mar 13 17:27:09 2010 New Revision: 157428 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157428 Log: Add testcase from PR43349. 2010-03-11 Sebas

[Bug tree-optimization/43354] [4.5 Regression] function �pofun2' from xplor-nih ICEs compiler at -O2 -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-13 17:27 --- Subject: Bug 43354 Author: spop Date: Sat Mar 13 17:27:15 2010 New Revision: 157429 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157429 Log: Fix PR43354: Correctly handle default definitions. 20

[Bug tree-optimization/43306] [4.5 Regression] ICE: in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:713 with -O1 -fstrict-overflow -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-03-13 17:35 --- Subject: Bug 43306 Author: spop Date: Sat Mar 13 17:34:59 2010 New Revision: 157434 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157434 Log: Fix PR43306: correct evolution_function_right_is_int

[Bug tree-optimization/43349] [graphite] very long compile time for gamess with -floop-interchange

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #7 from spop at gcc dot gnu dot org 2010-03-13 17:35 --- Subject: Bug 43349 Author: spop Date: Sat Mar 13 17:35:37 2010 New Revision: 157439 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157439 Log: Add testcase from PR43349. 2010-03-11 Sebas

[Bug tree-optimization/43354] [4.5 Regression] function �pofun2' from xplor-nih ICEs compiler at -O2 -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-13 17:35 --- Subject: Bug 43354 Author: spop Date: Sat Mar 13 17:35:45 2010 New Revision: 157440 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157440 Log: Fix PR43354: Correctly handle default definitions. 20

[Bug tree-optimization/43354] [4.5 Regression] function �pofun2' from xplor-nih ICEs compiler at -O2 -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-03-13 17:36 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug tree-optimization/43349] [graphite] very long compile time for gamess with -floop-interchange

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #8 from spop at gcc dot gnu dot org 2010-03-13 17:36 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug tree-optimization/43306] [4.5 Regression] ICE: in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:713 with -O1 -fstrict-overflow -fgraphite-identity

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #7 from spop at gcc dot gnu dot org 2010-03-13 17:43 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug tree-optimization/43351] ICE: SIGSEGV with -fgraphite-identity in 4.4

2010-03-13 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-13 17:49 --- Subject: Bug 43351 Author: spop Date: Sat Mar 13 17:49:24 2010 New Revision: 157442 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157442 Log: Add testcase for PR43351. 2010-03-13 Sebastian Pop

[Bug middle-end/43396] Very long compile time with -O1 -fgraphite-identity

2010-03-16 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-16 21:23 --- You may want to lower the --param graphite-max-nb-scop-params=5 the current value is at 10. Does this help? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43396

[Bug middle-end/43396] Very long compile time with -O1 -fgraphite-identity

2010-03-16 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-16 21:38 --- I think it is not due to the number of parameters: as 3 is pretty low. It might be the code that we generate for all the cases. I will give it a look. -- spop at gcc dot gnu dot org changed: What

[Bug c/43398] ICE with -O -floop-block

2010-03-17 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-17 15:13 --- This looks like a duplicate of PR42860. This works on gcc4.5. *** This bug has been marked as a duplicate of 42860 *** -- spop at gcc dot gnu dot org changed: What|Removed

[Bug middle-end/42860] ICE in gcc-4.4.3 with graphite

2010-03-17 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-17 15:13 --- *** Bug 43398 has been marked as a duplicate of this bug. *** -- spop at gcc dot gnu dot org changed: What|Removed |Added

[Bug middle-end/42860] ICE in gcc-4.4.3 with graphite

2010-03-17 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-17 15:14 --- See PR43398 for a nicely reduced testcase. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42860

[Bug tree-optimization/43423] gcc should vectorize this loop through "iteration range splitting"

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-18 18:51 --- Yes, I think we should improve if-conversion to handle more complex cases. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43423

[Bug tree-optimization/43427] The loop is not interchanged and thus could not be vectorized.

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-18 18:59 --- In the output of ./cc1 -O3 -floop-interchange -fdump-tree-graphite-all -ftree-vectorizer-verbose=7 we have: "Loops at depths 0 and 1 will be interchanged." so we do the interchange, but then the vectorizer

[Bug tree-optimization/43427] The loop is not interchanged and thus could not be vectorized.

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-18 19:01 --- ./cc1 -O3 -msse2 -ffast-math -ftree-vectorizer-verbose=2 pr43427.c -ftree-loop-linear pr43427.c:6: note: not vectorized: complicated access pattern. pr43427.c:7: note: LOOP VECTORIZED. pr43427.c:3: note: vectorized 1

[Bug tree-optimization/43430] New: Missed vectorization: "stmt not supported: cond_expr"

2010-03-18 Thread spop at gcc dot gnu dot org
ion AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: spop at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43430

[Bug tree-optimization/43431] New: Diagnostic message is not clear for vectorization profitability analysis

2010-03-18 Thread spop at gcc dot gnu dot org
The diagnostic message is not clear when compiling this code from FFmpeg with gcc-4.5 -c vsad_intra.c -O3 -ffast-math -ftree-vectorizer-verbose=7 -msse2 [...] ac3_downmix.c:10: note: cost model: vector iteration cost = 2060 is divisible by scalar iteration cost = 7 by a factor greater than or equa

[Bug tree-optimization/43432] New: Missed vectorization: "complicated access pattern" for increasing and decreasing data indexing

2010-03-18 Thread spop at gcc dot gnu dot org
Both these loop kernels fail with: gcc-4.5 -c vector_fmul_window.c -O3 -ffast-math -ftree-vectorizer-verbose=7 -msse2 [...] vector_fmul_window.c:7: note: not vectorized: complicated access pattern. They look similar in the fact that they use both an increasing and a decreasing induction variables

[Bug tree-optimization/43433] New: Missed vectorization: "complicated access pattern" for increasing and decreasing data indexing

2010-03-18 Thread spop at gcc dot gnu dot org
Both these loop kernels from FFmpeg fail with: gcc-4.5 -c vector_fmul_window.c -O3 -ffast-math -ftree-vectorizer-verbose=7 -msse2 [...] vector_fmul_window.c:7: note: not vectorized: complicated access pattern. They look similar in the fact that they use both an increasing and a decreasing inductio

[Bug tree-optimization/43433] Missed vectorization: "complicated access pattern" for increasing and decreasing data indexing

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-18 21:08 --- Duplicated of PR43432: pressed "commit bug" twice. *** This bug has been marked as a duplicate of 43432 *** -- spop at gcc dot gnu dot org changed: What|Removed

[Bug tree-optimization/43432] Missed vectorization: "complicated access pattern" for increasing and decreasing data indexing

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-18 21:08 --- *** Bug 43433 has been marked as a duplicate of this bug. *** -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43432

[Bug tree-optimization/43434] New: Missed vectorization: "not vectorized: data ref analysis": pointer incremented by a parameter

2010-03-18 Thread spop at gcc dot gnu dot org
Severity: normal Priority: P3 Component: tree-optimization AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: spop at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43434

[Bug tree-optimization/43436] New: Missed vectorization: "unhandled data-ref"

2010-03-18 Thread spop at gcc dot gnu dot org
This kernel from FFmpeg is not vectorized with: gcc-4.5 -c sub_hfyu_median_prediction.c -O3 -ffast-math -ftree-vectorizer-verbose=7 -msse2 [...] sub_hfyu_median_prediction.c:18: note: not vectorized: unhandled data-ref Looking with GDB at it, I get: (gdb) p debug_data_references (datarefs) (Data

[Bug tree-optimization/43436] Missed vectorization: "unhandled data-ref"

2010-03-18 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-18 22:04 --- Also note that a similar problem occurs for hadamard8: gcc-4.5 -c hadamard8.c -O3 -ffast-math -ftree-vectorizer-verbose=7 -msse2 [...] hadamard8_diff.c:44: note: not vectorized: unhandled data-ref hadamard8_diff.c:26

[Bug middle-end/43464] New: copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: spop at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43464

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-21 07:01 --- Created an attachment (id=20148) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20148&action=view) pr43464.i -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43464

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-21 07:06 --- Created an attachment (id=20149) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20149&action=view) proposed fix The proposed fix is to recompute the loop closed SSA form after copy propagation. --

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-21 07:32 --- Subject: Bug 43464 Author: spop Date: Sun Mar 21 07:32:43 2010 New Revision: 157602 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157602 Log: Fix PR43464: update loop close SSA once copy prop is don

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-03-21 17:01 --- Further reduced testcase: typedef struct regnode { char flags; } regnode; extern const unsigned char A[]; char *foo (regnode *c, char *s, int norun) { int uskip; while (s + (uskip = A[*s])) { if ((c

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #7 from spop at gcc dot gnu dot org 2010-03-21 17:17 --- The problem is that copyprop does this change: @@ -25,7 +25,7 @@ : # .MEM_16 = PHI <.MEM_18(10), .MEM_20(11)> # s_66 = PHI -s_13 = s_66; +s_13 = s_1; goto (got_it); }

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #10 from spop at gcc dot gnu dot org 2010-03-22 00:17 --- Yes, this patch does fix the problem. diff --git a/gcc/tree-ssa-copy.c b/gcc/tree-ssa-copy.c index 57c6558..61e32cc 100644 --- a/gcc/tree-ssa-copy.c +++ b/gcc/tree-ssa-copy.c @@ -797,7 +797,6 @@ init_copy_prop (void

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #11 from spop at gcc dot gnu dot org 2010-03-22 00:26 --- I would still like to see some extra checking after copyprop: would this extra check be ok to commit with the fix? diff --git a/gcc/tree-ssa-copy.c b/gcc/tree-ssa-copy.c index 61e32cc..011a80a 100644 --- a/gcc/tree

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-21 Thread spop at gcc dot gnu dot org
--- Comment #12 from spop at gcc dot gnu dot org 2010-03-22 01:29 --- Subject: Bug 43464 Author: spop Date: Mon Mar 22 01:28:51 2010 New Revision: 157617 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157617 Log: Fix PR43464: loop close phi nodes can contain more

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-22 Thread spop at gcc dot gnu dot org
--- Comment #13 from spop at gcc dot gnu dot org 2010-03-22 15:47 --- Note that * tree-ssa-copy.c (init_copy_prop): Loop closed phi nodes can contain more than one argument. breaks both 464.h264ref and 416.gamess with -O3. See for example http://groups.google.com

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-23 Thread spop at gcc dot gnu dot org
--- Comment #27 from spop at gcc dot gnu dot org 2010-03-23 20:08 --- I just verified that the problem is well in CLooG-PPL. With CLooG-ISL we generate a correct code that looks like this: for (c2=1;c2<=D.1554_10-3;c2++) { S1(c2); for (c4=0;c4<=c2-1;c4++) { S2

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-23 Thread spop at gcc dot gnu dot org
--- Comment #28 from spop at gcc dot gnu dot org 2010-03-23 21:24 --- I fixed this problem in CLooG-PPL: the code generated with the new version looks like the code generated by CLooG-ISL: for (c2=1;c2<=D.1554_10-3;c2++) { S1(c2); for (c4=0;c4<=c2-1;c4++) { S2

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-23 Thread spop at gcc dot gnu dot org
--- Comment #29 from spop at gcc dot gnu dot org 2010-03-23 21:27 --- Also note that with the fix in CLooG-PPL, gfortran -O2 -fgraphite-identity air.f90 -o air ./air passes without error. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42181

[Bug middle-end/42181] [4.5 Regression][graphite] -fgraphite-identity miscompiles air.f90

2010-03-24 Thread spop at gcc dot gnu dot org
--- Comment #30 from spop at gcc dot gnu dot org 2010-03-24 20:17 --- Fixed. Please use ftp://gcc.gnu.org/pub/gcc/infrastructure/cloog-ppl-0.15.9.tar.gz -- spop at gcc dot gnu dot org changed: What|Removed |Added

[Bug middle-end/42777] [graphite] loop interchange heuristic is broken

2010-03-24 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-24 21:11 --- I don't think this is due to the interchange heuristic, it looks more like a missed vectorization problem after Graphite. Closing the bug. -- spop at gcc dot gnu dot org changed: What|Re

[Bug middle-end/42130] [graphite-branch] dealII fails

2010-03-25 Thread spop at gcc dot gnu dot org
--- Comment #12 from spop at gcc dot gnu dot org 2010-03-25 15:15 --- The patch for the fix is: http://gcc.gnu.org/viewcvs?view=revision&revision=157286 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42130

[Bug middle-end/43519] New: [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-03-25 Thread spop at gcc dot gnu dot org
Java libs Product: gcc Version: 4.5.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: spop at gcc dot gnu dot org

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-26 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-03-26 20:08 --- Due to the "FORNOW" fail in vectorizable_condition: int nunits = TYPE_VECTOR_SUBPARTS (vectype); int ncopies = LOOP_VINFO_VECT_FACTOR (loop_vinfo) / nunits; if (ncopies > 1) return fal

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-26 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-26 20:13 --- Reduced testcase, for which ncopies is 4. typedef unsigned char uint8_t; vsad16_c (void *c, uint8_t * s1, uint8_t * s2, int stride, int h) { int score = 0; int x, y; for (x = 0; x < 16; x++) score += ((s

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-26 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-26 20:26 --- vectorizable_condition has to be cleaned up of this condition: /* We do not handle two different vector types for the condition and the values. */ if (TREE_TYPE (TREE_OPERAND (cond_expr, 0)) != TREE_TYPE

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-26 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-26 20:49 --- This cleanup of vectorizable_condition allows this testcase to be vectorized: typedef int uint8_t; vsad16_c (void *c, uint8_t * s1, uint8_t * s2, int stride, int h) { int score = 0; int x, y; for (x = 0; x <

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-26 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-26 21:18 --- Created an attachment (id=20218) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20218&action=view) Cleanup comparisons of types This patch replaces with types_compatible_p all the places in the vectoriz

[Bug tree-optimization/43431] Diagnostic message is not clear for vectorization profitability analysis

2010-03-28 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-03-28 15:56 --- (In reply to comment #1) > > What does this message mean? > > "vector iteration cost = 2056 is divisible by scalar iteration cost = 4 by a > > factor greater than or equal to the vectorization f

[Bug tree-optimization/43436] Missed vectorization: "unhandled data-ref"

2010-03-28 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-28 16:28 --- What about fixing the diagnostic message like this: diff --git a/gcc/tree-vect-data-refs.c b/gcc/tree-vect-data-refs.c index 37ae9b5..44248b3 100644 --- a/gcc/tree-vect-data-refs.c +++ b/gcc/tree-vect-data-refs.c

[Bug tree-optimization/43436] Missed vectorization: "unhandled data-ref"

2010-03-28 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-03-28 16:35 --- When defining the missing function like this: static inline int mid_pred(int a, int b, int c) { int t= (a-b)&((a-b)>>31); a-=t; b+=t; b-= (b-c)&((b-c)>>31); b+= (a-b)&((

[Bug tree-optimization/43431] Diagnostic message is not clear for vectorization profitability analysis

2010-03-28 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2010-03-28 16:57 --- What about fixing this bug with diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c index afbd342..2601b58 100644 --- a/gcc/tree-vect-loop.c +++ b/gcc/tree-vect-loop.c @@ -2173,9 +2173,9

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-30 Thread spop at gcc dot gnu dot org
--- Comment #14 from spop at gcc dot gnu dot org 2010-03-30 16:57 --- Subject: Bug 43464 Author: spop Date: Tue Mar 30 16:56:49 2010 New Revision: 157828 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157828 Log: Fix PR43464: copyprop should maintain loop close phi no

[Bug tree-optimization/43430] Missed vectorization: "stmt not supported: cond_expr"

2010-03-30 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-03-30 19:58 --- Subject: Bug 43430 Author: spop Date: Tue Mar 30 19:58:35 2010 New Revision: 157833 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157833 Log: Replace type != type comparisons with types_compatible_p.

[Bug tree-optimization/43351] ICE: SIGSEGV with -fgraphite-identity in 4.4

2010-03-31 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-03-31 18:37 --- Subject: Bug 43351 Author: spop Date: Wed Mar 31 18:36:45 2010 New Revision: 157882 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157882 Log: Add testcase for PR43351. 2010-03-13 Sebastian Pop

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-31 Thread spop at gcc dot gnu dot org
--- Comment #15 from spop at gcc dot gnu dot org 2010-03-31 18:38 --- Subject: Bug 43464 Author: spop Date: Wed Mar 31 18:37:41 2010 New Revision: 157889 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157889 Log: Fix PR43464: copyprop should maintain loop close phi no

[Bug middle-end/43464] copy prop breaks loop closed SSA form

2010-03-31 Thread spop at gcc dot gnu dot org
--- Comment #16 from spop at gcc dot gnu dot org 2010-03-31 18:48 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|NEW

[Bug tree-optimization/43427] The loop is not interchanged and thus could not be vectorized.

2010-04-01 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2010-04-01 18:59 --- I think that this bug should be closed as fixed, unless we change the subject to: "At -O3 the loop is not interchanged and thus not vectorized". -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43427

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-05 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-04-05 06:59 --- Subject: Bug 43519 Author: spop Date: Mon Apr 5 06:58:46 2010 New Revision: 157963 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157963 Log: Compute min and max bounds for IVs and infer types. 20

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-05 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-04-06 03:16 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 03:15:58 2010 New Revision: 157976 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157976 Log: Use POINTER_PLUS_EXPR for pointer types. 2010-04-04 S

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-05 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-04-06 03:16 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 03:15:58 2010 New Revision: 157976 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157976 Log: Use POINTER_PLUS_EXPR for pointer types. 2010-04-04 S

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-05 Thread spop at gcc dot gnu dot org
--- Comment #8 from spop at gcc dot gnu dot org 2010-04-06 06:14 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 06:14:26 2010 New Revision: 157978 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157978 Log: Fix 473.astar miscompile. 2010-04-05 Sebastian Pop

[Bug target/43667] ICE with __builtin_ia32_vpermil2p* and -mavx

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2010-04-06 15:45 --- The fix looks good. Are you testing this patch, or should I ask Quentin to extra test it on our end? Thanks, Sebastian -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43667

[Bug tree-optimization/32824] Missed reduction vectorizer after store to global is LIM'd

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #9 from spop at gcc dot gnu dot org 2010-04-06 19:20 --- Subject: Bug 32824 Author: spop Date: Tue Apr 6 19:19:41 2010 New Revision: 158018 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158018 Log: Fix PR32824. 2010-04-06 Changpeng Fang PR mi

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #9 from spop at gcc dot gnu dot org 2010-04-06 21:01 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 21:01:16 2010 New Revision: 158026 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158026 Log: Compute min and max bounds for IVs and infer types. 20

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #10 from spop at gcc dot gnu dot org 2010-04-06 21:02 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #11 from spop at gcc dot gnu dot org 2010-04-06 21:03 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 21:02:25 2010 New Revision: 158027 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158027 Log: Use POINTER_PLUS_EXPR for pointer types. 2010-04-04 S

[Bug middle-end/43519] [graphite] Bootstrap with Graphite enabled fails in Java libs

2010-04-06 Thread spop at gcc dot gnu dot org
--- Comment #12 from spop at gcc dot gnu dot org 2010-04-06 21:03 --- Subject: Bug 43519 Author: spop Date: Tue Apr 6 21:03:37 2010 New Revision: 158028 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158028 Log: Fix 473.astar miscompile. 2010-04-05 Sebas

[Bug tree-optimization/43427] The loop is not interchanged and thus could not be vectorized.

2010-04-08 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2010-04-08 17:40 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|NEW

[Bug tree-optimization/43423] gcc should vectorize this loop through if-conversion

2010-04-08 Thread spop at gcc dot gnu dot org
--- Comment #6 from spop at gcc dot gnu dot org 2010-04-08 17:47 --- I changed the title of this bug to match the comments in the PR: we should vectorize this loop using if-conversion, and not "iteration range splitting". Also note that in general, by doing an "

[Bug tree-optimization/32824] Missed reduction vectorizer after store to global is LIM'd

2010-04-10 Thread spop at gcc dot gnu dot org
--- Comment #10 from spop at gcc dot gnu dot org 2010-04-11 05:45 --- Fixed. -- spop at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED

[Bug bootstrap/43737] New: Bootstrap broken at -O3

2010-04-12 Thread spop at gcc dot gnu dot org
strap AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: spop at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43737

[Bug bootstrap/43737] Bootstrap broken at -O3

2010-04-12 Thread spop at gcc dot gnu dot org
--- Comment #1 from spop at gcc dot gnu dot org 2010-04-13 05:41 --- This commit causes the bootstrap to fail at -O3. c26ce8a90a7c8ef9b6587959083d12f6edbc5e01 is first bad commit commit c26ce8a90a7c8ef9b6587959083d12f6edbc5e01 Author: rguenth Date: Wed Apr 7 12:31:32 2010 +

[Bug middle-end/42050] [4.5 Regression] ice in graphite-clast-to-gimple.c:165

2009-11-19 Thread spop at gcc dot gnu dot org
--- Comment #2 from spop at gcc dot gnu dot org 2009-11-19 21:12 --- Fixed in the Graphite branch. The changes of the branch will be pushed into trunk soon. I will commit the reduced testcase to the Graphite testsuite. Sebastian -- spop at gcc dot gnu dot org changed

[Bug middle-end/40281] [4.4/4.5 Regression] -fprefetch-loop-arrays: ICE: in initialize_matrix_A, at tree-data-ref.c:1887

2009-11-19 Thread spop at gcc dot gnu dot org
--- Comment #8 from spop at gcc dot gnu dot org 2009-11-19 22:14 --- Subject: Re: [4.4/4.5 Regression] -fprefetch-loop-arrays: ICE: in initialize_matrix_A, at tree-data-ref.c:1887 Hi, Here is a fix for this one, on top of the graphite branch. I will commit this fix to the

[Bug middle-end/42050] [4.5 Regression] ice in graphite-clast-to-gimple.c:165

2009-11-19 Thread spop at gcc dot gnu dot org
--- Comment #3 from spop at gcc dot gnu dot org 2009-11-19 22:33 --- Subject: Bug 42050 Author: spop Date: Thu Nov 19 22:32:44 2009 New Revision: 154347 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154347 Log: Testcase for PR42050. 2009-11-18 Sebastian Pop

[Bug middle-end/40281] [4.4/4.5 Regression] -fprefetch-loop-arrays: ICE: in initialize_matrix_A, at tree-data-ref.c:1887

2009-11-19 Thread spop at gcc dot gnu dot org
--- Comment #9 from spop at gcc dot gnu dot org 2009-11-19 22:33 --- Subject: Bug 40281 Author: spop Date: Thu Nov 19 22:32:50 2009 New Revision: 154348 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154348 Log: Fix PR40281. 2009-11-18 Sebastian Pop PR mi

[Bug middle-end/41924] graphite miscompiles aermod

2009-11-21 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2009-11-21 09:04 --- Yes, this should be fixed now in the Graphite branch, and it will be fixed in trunk after the merge that I am preparing. Sebastian -- spop at gcc dot gnu dot org changed: What|Removed

[Bug middle-end/41924] graphite miscompiles aermod

2009-11-21 Thread spop at gcc dot gnu dot org
--- Comment #5 from spop at gcc dot gnu dot org 2009-11-21 09:09 --- My automatic testers are still failing one of the aermod tests, see for example pb05.sum from: http://groups.google.com/group/gcc-graphite-test/browse_thread/thread/4afb9c19aa8a10c7 -- spop at gcc dot gnu dot org

[Bug tree-optimization/41118] Wrong dependence analysis in graphite for unrestricted pointers

2009-11-24 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2009-11-25 04:49 --- Subject: Bug 41118 Author: spop Date: Wed Nov 25 04:48:51 2009 New Revision: 154549 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154549 Log: 2009-09-17 Li Feng PR middle-e

[Bug middle-end/42050] [4.5 Regression] ice in graphite-clast-to-gimple.c:165

2009-11-24 Thread spop at gcc dot gnu dot org
--- Comment #4 from spop at gcc dot gnu dot org 2009-11-25 05:32 --- Subject: Bug 42050 Author: spop Date: Wed Nov 25 05:32:03 2009 New Revision: 154640 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154640 Log: 2009-11-18 Sebastian Pop PR middle-e

[Bug middle-end/40281] [4.4/4.5 Regression] -fprefetch-loop-arrays: ICE: in initialize_matrix_A, at tree-data-ref.c:1887

2009-11-24 Thread spop at gcc dot gnu dot org
--- Comment #10 from spop at gcc dot gnu dot org 2009-11-25 05:36 --- Subject: Bug 40281 Author: spop Date: Wed Nov 25 05:35:58 2009 New Revision: 154641 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154641 Log: 2009-11-18 Sebastian Pop PR middle-e

<    1   2   3   4   5   6   7   8   >