RE: [PATCH] [MIPS] Support microMIPS HI/QI moves

2013-05-01 Thread Maciej W. Rozycki
On Mon, 22 Apr 2013, Moore, Catherine wrote: > > Dumb question: what's the difference between "all" and "*" in the > > compression attribute? > > The port defines the compression and enabled attributes like this: > > (define_attr "compression" "none,all,micromips" > (const_string "none")) >

RE: [PATCH] [MIPS] Support microMIPS HI/QI moves

2013-04-22 Thread Moore, Catherine
> -Original Message- > From: Maciej W. Rozycki [mailto:ma...@codesourcery.com] > Sent: Monday, April 22, 2013 5:09 PM > To: Moore, Catherine > Cc: rdsandif...@googlemail.com; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] [MIPS] Support microMIPS HI/QI moves > > O

Re: [PATCH] [MIPS] Support microMIPS HI/QI moves

2013-04-22 Thread Maciej W. Rozycki
On Mon, 1 Apr 2013, Moore, Catherine wrote: > (define_insn "*movhi_internal" > - [(set (match_operand:HI 0 "nonimmediate_operand" "=d,d,d,m,*a,*d") > - (match_operand:HI 1 "move_operand" "d,I,m,dJ,*d*J,*a"))] > + [(set (match_operand:HI 0 "nonimmediate_operand" "=d,!u,d,!u,d,ZU,m,

Re: [PATCH] [MIPS] Support microMIPS HI/QI moves

2013-04-22 Thread Richard Sandiford
"Moore, Catherine" writes: > 2013-04-01 Catherine Moore > > * config/mips/mips.md (*movhi_internal, *movqi_internal): New > operands. Record compression. OK, thanks. Note that LBU and LHU also appear in *zero_extend2, *zero_extendqihi2 and *and3. Richard