Re: [Patch] New: CR16 port

[Rask Ingemann Lambertsen <rask@sygehus.dk>]

On Tue, Jul 10, 2007 at 06:45:55AM +0000, Gadad, Pompapathi V wrote:

> Index: ChangeLog
> ===================================================================
> --- ChangeLog	(revision 126478)
> +++ ChangeLog	(working copy)
> @@ -1,3 +1,24 @@

   Don't include the ChangeLog entry in the patch, post it separately. The
only exception is if you've already committed the patch. Then it is ok to
include the ChangeLog entry as part of the patch.

> +;;  Negation Instructions
> +; move 0,r sub x,r. More efficient than xor -1,x add 1,x
> +(define_expand "neg<mode>2"
> +  [(set (match_operand:CR16IM 0 "register_operand" "")
> +        (const_int 0))
> +   (parallel [(set (match_dup 0) 
> +        (minus:CR16IM (match_dup 0)
> +          (match_operand:CR16IM 1 "register_operand" "")))
> +    (clobber (reg:CC CC_REGNUM))]) 
> +  ]
> +  ""
> +  "")

   You can probably leave this out as the middle end will generate "op0 = 0
- op1" if there's no negm2 pattern.

> +(define_insn "mov<mode>_regs"
> +  [(set (match_operand:ALLMT 0 "register_operand" "=r, r")
> +    (match_operand:ALLMT 1 "nonmemory_operand" "r, <iF>"))]
> +  ""
> +  "mov<tIsa>\t%1, %0"
> +  [(set_attr "length" "2,<lImmArith>")]
> +)
> +
> +; Memory => Reg
> +(define_insn "mov<mode>_load"
> +  [(set (match_operand:ALLMT 0 "register_operand" "=r")
> +    (match_operand:ALLMT 1 "memory_operand" "m"))]
> +  ""
> +  "load<tIsa>\t%1, %0"
> +  [(set_attr "length" "<lImmArith>")]
> +)
> +
> +(define_insn "mov<mode>_store_reg"
> +  [(set (match_operand:ALLMT 0 "store_operand" "=m")
> +    (match_operand:ALLMT 1 "register_operand" "r"))]
> +  ""
> +  "stor<tIsa>\t%1, %0"
> +  [(set_attr "length" "<lImmArith>")]
> +)
> +
> +(define_insn "mov<mode>_store_imm4"
> +  [(set (match_operand:SHORT 0 "store_operand" "=m")
> +    (match_operand:SHORT 1 "u4bits_operand" "<LL>"))]
> +  ""
> +  "stor<tIsaShort>\t%1, %0"
> +  [(set_attr "length" "<lImmShort>")]
> +)

   These would normally be one pattern with five alternatives. A reg->reg
copy can turn into a reg->mem copy during reload, for example.

-- 
Rask Ingemann Lambertsen