------- Comment #2 from hjl at lucon dot org 2007-04-05 07:35 ------- (In reply to comment #1) > (In reply to comment #0) > > > 1. Is ix86_binary_operator_ok needed here? > > Yes, it prevents expander and combiner to create two mem operands (please note > that reload can also resolve this case by itself, but some CSE opportunities > can be missed). >
There is no corresponding define_expand for this pattern. Many define_insn patterns without define_expand don't call ix86_binary_operator_ok. Will that be a problem? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31478