This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Powerpc patch #5, Add power7/vsx support (fix bugs in patch #4)
- From: Michael Meissner <meissner at linux dot vnet dot ibm dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>, David Edelsohn <dje at watson dot ibm dot com>
- Date: Wed, 29 Jul 2009 16:14:32 -0400
- Subject: Re: [PATCH] Powerpc patch #5, Add power7/vsx support (fix bugs in patch #4)
- References: <4A6FF16A.4080606@gmail.com>
On Wed, Jul 29, 2009 at 08:51:22AM +0200, Uros Bizjak wrote:
> Hello!
>
> >This patch is a respin of patch #4, that fixes the V2DImode problem on
> >altivec
> >that I noticed after submitting the patch, and to address the comments
> >David
> >made about patch #4 (using CONST0_RTX instead of const_int 0, and changing
> >the
> >UNSPEC number so it doesn't collide with rs6000.md).
> >
> >
>
> You don't need to handle BUILT_IN_{FABS, FMIN, FMAX}{,F} in target
> dependant vectorized_function. These builtins are already handled by
> generic code and will never reach this function as long as you have
> correct named patterns in the .md file.
Thanks. Yes, I figured that was the case. I'll remove them in the next
patch.
--
Michael Meissner, IBM
4 Technology Place Drive, MS 2203A, Westford, MA, 01886, USA
meissner@linux.vnet.ibm.com