On Wed, Oct 24, 2012 at 3:01 PM, Andrey Turetskiy <andrey.turets...@gmail.com> wrote:
> On Tue, Oct 23, 2012 at 2:45 PM, Andrey Turetskiy > <andrey.turets...@gmail.com> wrote: >> Hi, >> >> This patch replaces large const_vector constructions with >> match_operand in sse.md to decrease its size. >> Is it ok? No, you don't have to touch generic expand machinery. In the expander, use (match_dup X), and declare "operands[X] = CONST1_RTX (..some_mode..)" in preparation statement. In unnamed patterns, use const1_operand operand predicate. You should extend existing const1_operand in the same way as const0_operand. This approach is not compatible with named insn patterns, which duplicate its functionality as pattern matcher and as an expander. Uros.