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.

Reply via email to