This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: x86_64 merger part 39 - vardic functions..
- To: Jan Hubicka <jh at suse dot cz>
- Subject: Re: x86_64 merger part 39 - vardic functions..
- From: Richard Henderson <rth at redhat dot com>
- Date: Thu, 20 Sep 2001 14:57:16 -0700
- Cc: gcc-patches at gcc dot gnu dot org, patches at x86-64 dot org
- References: <20010920144930.A32741@atrey.karlin.mff.cuni.cz>
On Thu, Sep 20, 2001 at 02:49:30PM +0200, Jan Hubicka wrote:
> here comes the x86_64 vardic function stuff.
s/vardic/variadic/g
> ! ix86_save_varrargs_registers = 1;
varargs
> + (define_expand "sse_prologue_save"
Why do you bother with an expander?
> + (define_insn "*sse_prologue_save_insn"
> + [(set (mem:BLK (plus:DI (match_operand:DI 0 "register_operand" "R")
> + (match_operand:DI 4 "const_int_operand" "n")))
> + (unspec:BLK [(clobber (reg:DI 21))
You're not clobbering the SSE registers.
I'm surprised you do a computed jump like this instead of
just saving all of the SSE registers that might contain data.
r~