This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: Your change to emit_group_{load,store}
- From: Richard Henderson <rth at redhat dot com>
- To: Aldy Hernandez <aldyh at redhat dot com>
- Cc: Richard Kenner <kenner at vlsi1 dot ultra dot nyu dot edu>, gcc at gcc dot gnu dot org
- Date: Tue, 26 Oct 2004 11:53:49 -0700
- Subject: Re: Your change to emit_group_{load,store}
- References: <10410261754.AA21817@vlsi1.ultra.nyu.edu> <20041026175826.GA2077@redhat.com>
On Tue, Oct 26, 2004 at 01:58:26PM -0400, Aldy Hernandez wrote:
> On Tue, Oct 26, 2004 at 01:54:05PM -0400, Richard Kenner wrote:
> > I don't understand the test for !SCALAR_INT_MODE_P. That include BLKmode,
> > but you call emit_move_insn in the new code, which won't work for BLKmode.
> > What did you mean here? Perhaps SCALAR_FLOAT_MODE_P?
> >
> > The test case is compiling the Ada RTS file a-calend.adb with -O2 on x864-64.
>
> Hmmm... perhaps what we want is SCALAR_FLOAT_MODE_P or
> (!SCALAR_INT_MODE_P && !BLKmode).
The later I would think.
> (Further hmmmm... I can't recall wether this affected complex floats
> or not.
For your target, I dunno. But clearly emit_group_load/store can't
handle anything but scalar integers or blkmode, so...
r~