On Fri, Oct 21, 2011 at 03:44:11PM +0200, Ira Rosen wrote: > But it's OK to allow modifier != NONE if it's not SLP, so we need &&, no?
Well, in my patch that check was guarded by the if (slp_node ...), so presumably it would allow modifier == NARROW vectorization in the loops (otherwise some testcases would fail I'd hope). Is gcc_assert ((slp_node != NULL) == (STMT_SLP_TYPE (stmt_info) != 0)); always true? If not, when it is not? When would be slp_node == NULL and PURE_SLP_STMT true? Jakub