This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH/RFA] SH: Rename ashlhi3 insn


On Mar 21, 2005, Kaz Kojima <kkojima@rr.iij4u.or.jp> wrote:

> 	* config/sh/sh.md (ashlhi3): Rename to *ashlhi3_n and add a new
> 	ashlhi3 expander.

> +  if (GET_CODE (operands[2]) != CONST_INT || operands[0] != operands[1])
> +    FAIL;

I don't think the requirement that operands[0] == operands[1] is
appropriate.  The register allocator could arrange for input and
output pseudos to be assigned to the same hardware register.  If you
absolutely need this test, add `&& reload_completed', otherwise just
leave it out.

-- 
Alexandre Oliva             http://www.ic.unicamp.br/~oliva/
Red Hat Compiler Engineer   aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist  oliva@{lsd.ic.unicamp.br, gnu.org}


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]