This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Last alignment change for MEM tracking
- From: Graham Stott <grahams at redhat dot com>
- To: Richard Kenner <kenner at vlsi1 dot ultra dot nyu dot edu>
- Cc: bryce at waitaki dot otago dot ac dot nz, gcc-patches at gcc dot gnu dot org
- Date: Fri, 16 Nov 2001 08:01:02 -0500
- Subject: Re: Last alignment change for MEM tracking
- References: <10111161124.AA20809@vlsi1.ultra.nyu.edu>
Richard,
Richard Kenner wrote:
>
> A couple of occurances in .md files got missed. I'm checking in the
> following as obvious.
>
> Thanks. I thought I grepped config/*/*, but I guess I did
> config/*/*.c. Sorry.
>
> Still got the i686 bootstrap failure.
>
> Can you send me the .i file?
One problem I've noticed in optabs.c is the calls to gen_cmpstrxx are
passing the wrong argument type for the 5th paramater which requires
a (rtx) but is being passed an (unsigned int).
Unfortunately this still doesn't get us back to bootstrap land on x86.
Graham
ChangeLog
* optabs.c (prepare_cmp_insn): Correct lossage in last change
OPALIGN type.
Index: optabs.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/optabs.c,v
retrieving revision 1.117
diff -c -p -r1.117 optabs.c
*** optabs.c 2001/11/15 23:44:57 1.117
--- optabs.c 2001/11/16 12:03:46
*************** prepare_cmp_insn (px, py, pcomparison, s
*** 3101,3108 ****
{
rtx result;
enum machine_mode result_mode;
! unsigned int opalign ATTRIBUTE_UNUSED
! = (MIN (MEM_ALIGN (x), MEM_ALIGN (y)) / BITS_PER_UNIT);
emit_queue ();
x = protect_from_queue (x, 0);
--- 3101,3108 ----
{
rtx result;
enum machine_mode result_mode;
! rtx opalign ATTRIBUTE_UNUSED
! = GEN_INT (MIN (MEM_ALIGN (x), MEM_ALIGN (y)) / BITS_PER_UNIT);
emit_queue ();
x = protect_from_queue (x, 0);