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] Another fix for powerpc-linux-gnualtivec

From: Aldy Hernandez <aldyh at redhat dot com>
To: Alan Matsuoka <alanm at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: 21 Feb 2002 10:10:56 +1100
Subject: Re: [PATCH] Another fix for powerpc-linux-gnualtivec
References: <20020220051444.GB26044@cpe0050daeb94fe.cpe.net.cable.rogers.com>

>>>>> "Alan" == Alan Matsuoka <alanm@redhat.com> writes:

 > *************** vrsave_operation (op, mode)
 > *** 5324,5330 ****
 >       return 0;

 >     dest_regno = REGNO (SET_DEST (XVECEXP (op, 0, 0)));
 > !   src_regno  = REGNO (SET_SRC (XVECEXP (op, 0, 0)));

 >     if (dest_regno != VRSAVE_REGNO
 >         && src_regno != VRSAVE_REGNO)
 > --- 5324,5330 ----
 >       return 0;

 >     dest_regno = REGNO (SET_DEST (XVECEXP (op, 0, 0)));
 > !   src_regno  = REGNO (XVECEXP (XEXP (XVECEXP (op, 0, 0), 1), 0, 0));

 >     if (dest_regno != VRSAVE_REGNO
 >         && src_regno != VRSAVE_REGNO)

there should be a prettier macro to use here, though i don't know what
it is.  This "(XVECEXP (XEXP (XVECEXP" stuff looks pretty hairy.  The
idea is correct though.

Perhaps separate it out into various rtl, checking that you have what
you expect along the way.

Take richard's advices as well.

References:
- [PATCH] Another fix for powerpc-linux-gnualtivec
  - From: Alan Matsuoka

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