This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: x86-64 merger part 9 - few cleanups/fixes
- To: Jan Hubicka <jh at suse dot cz>
- Subject: Re: x86-64 merger part 9 - few cleanups/fixes
- From: Richard Henderson <rth at redhat dot com>
- Date: Sat, 10 Mar 2001 13:40:53 -0800
- Cc: patches at x86-64 dot org, gcc-patches at gcc dot gnu dot org
- References: <20010310143311.E25748@atrey.karlin.mff.cuni.cz>
On Sat, Mar 10, 2001 at 02:33:11PM +0100, Jan Hubicka wrote:
> (define_expand "ashrdi3"
[...]
> + if (TARGET_64BIT)
> + {
> + ix86_expand_binary_operator (ASHIFTRT, DImode, operands); DONE;
> + }
Why is this one different from the other two? We hadn't been doing
ix86_expand_binary_operator on these, but perhaps we should have been.
> - {cltd|cdq}
> + {cltd|cdqe}
Surely this is wrong.
{"cltd", 0, 0x99, X, 0, NoSuf|Size32, { 0, 0, 0} },
{"cdq", 0, 0x99, X, 0, NoSuf|Size32, { 0, 0, 0} },
{"cdqe", 0, 0x98, X, Cpu64, NoSuf|Size64, { 0, 0, 0} },
One of these is not like the other.
Otherwise ok.
r~