------- Comment #9 from ubizjak at gmail dot com 2007-04-05 19:50 ------- (In reply to comment #8)
> > Please also change operands 1 and 2 of sdot_prodv8hi expander to > > register_operand to avoid further suprises. > > > > I am not sure if there is an issue since op0 of sse2_pmaddwd must be > register and it must be the same as op1 or op2. But only after reload. Nothing prevents gcc to generate two MEM input operands. Although reload will fix this by putting one of input operands into register (and match it with output operand), but this will be too late for any pre-reload pass to make any optimization on inputs. > is ix86_binary_operator_ok in sse2_umulv2siv2di3 not used at all and really > optional? No, it is not optional. V8HImode argument is optional, since we don't allow immediate operands. Please track code flow through ix86_b_o_ok() and you will see. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31478
