Re: [PATCH] Fixing PR60000: A bug in the vectorizer.

2014-01-31 Thread Richard Biener
On Thu, 30 Jan 2014, Cong Hou wrote: Hi PR6 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=6) is caused by GCC vectorizer. The bug appears when handling vectorization patterns. When a pattern statement has additional new statements stored in pattern_def_seq in vect_transform_loop(),

Re: [PATCH] Fixing PR60000: A bug in the vectorizer.

2014-01-31 Thread Jakub Jelinek
On Fri, Jan 31, 2014 at 09:41:59AM +0100, Richard Biener wrote: Is that because si and pattern_def_si point to the same stmts? Then I'd prefer to do if (is_store) { ... pattern_def_seq = NULL; } else if (!transform_pattern_stmt gsi_end_p (pattern_def_si)) {

Re: [PATCH] Fixing PR60000: A bug in the vectorizer.

2014-01-31 Thread Cong Hou
On Fri, Jan 31, 2014 at 5:06 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Jan 31, 2014 at 09:41:59AM +0100, Richard Biener wrote: Is that because si and pattern_def_si point to the same stmts? Then I'd prefer to do if (is_store) { ... pattern_def_seq = NULL;

Re: [PATCH] Fixing PR60000: A bug in the vectorizer.

2014-01-31 Thread Richard Biener
On February 1, 2014 12:06:55 AM GMT+01:00, Cong Hou co...@google.com wrote: On Fri, Jan 31, 2014 at 5:06 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Jan 31, 2014 at 09:41:59AM +0100, Richard Biener wrote: Is that because si and pattern_def_si point to the same stmts? Then I'd prefer

Re: [PATCH] Fixing PR60000: A bug in the vectorizer.

2014-01-30 Thread Cong Hou
Wrong format. Send it again. On Thu, Jan 30, 2014 at 4:57 PM, Cong Hou co...@google.com wrote: Hi PR6 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=6) is caused by GCC vectorizer. The bug appears when handling vectorization patterns. When a pattern statement has additional new