This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: Ping Re: Handle Octeon 3 not supporting MIPS paired-single instructions
- From: "Moore, Catherine" <Catherine_Moore at mentor dot com>
- To: "Myers, Joseph" <joseph_myers at mentor dot com>, Andrew Pinski <andrew dot pinski at caviumnetworks dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, "matthew dot fortune at imgtec dot com" <matthew dot fortune at imgtec dot com>
- Date: Sat, 16 Jan 2016 04:40:39 +0000
- Subject: RE: Ping Re: Handle Octeon 3 not supporting MIPS paired-single instructions
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot DEB dot 2 dot 10 dot 1601090005190 dot 22470 at digraph dot polyomino dot org dot uk> <CA+=Sn1n3msv_gjo2MKxGV=M=73y3OCd_6OfggJXkUCjzUEMATQ at mail dot gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1601152327130 dot 29672 at digraph dot polyomino dot org dot uk>
> -----Original Message-----
> From: Myers, Joseph
> Sent: Friday, January 15, 2016 6:28 PM
> To: Andrew Pinski
> Cc: GCC Patches; Moore, Catherine; matthew.fortune@imgtec.com
> Subject: Ping Re: Handle Octeon 3 not supporting MIPS paired-single
> instructions
>
> On Fri, 8 Jan 2016, Andrew Pinski wrote:
>
> > On Fri, Jan 8, 2016 at 4:05 PM, Joseph Myers <joseph@codesourcery.com>
> wrote:
> > > The Octeon 3 processor does not support the MIPS paired-single
> > > instructions. This results in illegal instruction errors in the
> > > testsuite when vectorization tests try to use those instructions.
> > >
> > > This patch teaches the compiler about that lack of support, so that
> > > warnings are given when -mpaired-single (or something implying it)
> > > is used when compiling for such a processor. I chose to test
> > > TARGET_OCTEON as the simplest conditional; since the older Octeon
> > > processors don't support hard float at all, I don't think the choice
> > > matters for them. Tests that then failed with the warning were
> > > updated to disable them for Octeon.
> > >
> > > Tested with no regressions for cross to mips64el-linux-gnu (Octeon
> > > 3). OK to commit?
> >
> > This is ok from my point of view. I did not think about doing this at
> > the time I added Octeon 3 support.
>
> Ping for MIPS maintainer review of this patch <https://gcc.gnu.org/ml/gcc-
> patches/2016-01/msg00492.html>.
>
This is OK, please commit.
Thanks,
Catherine