From: Segher Boessenkool <segher@kernel.crashing.org>
To: Michael Meissner <meissner@linux.vnet.ibm.com>,
gcc-patches@gcc.gnu.org,
David Edelsohn <dje.gcc@gmail.com>,
Bill Schmidt <wschmidt@linux.vnet.ibm.com>
Subject: Re: [PATCH], Patch #7, Add PowerPC vector initialization tests
Date: Sat, 27 Aug 2016 01:24:00 -0000 [thread overview]
Message-ID: <20160827012439.GA20847@gate.crashing.org> (raw)
In-Reply-To: <20160826192950.GA5763@ibm-tiger.the-meissners.org>
On Fri, Aug 26, 2016 at 03:29:50PM -0400, Michael Meissner wrote:
> These patches add more tests to the PowerPC vector initialization tests. Four
> of the tests added (#4, #5, #8, and #9) just try to do a bunch of vector
> initializations for the different vector type (char, short, float, and double).
>
> These other patches (#6, #7) test the code generated in paches #5 and #6.
>
> I have run tese tests on a big endian power7 system (with both 32-bt and 64-bit
> tests runs), a big endian power8 system (just 64-bit tests), and a little
> endian power8 system. There were no regressions. As these tests ok to
> install?
This is okay for trunk; one comment:
> +/* { dg-final { scan-assembler "mtvsrd" } } */
This also matches mtvsrdd; if you don't want that, you can avoid it by
writing it as {\mmtvsrd\M} (the {} instead of "" to avoid toothpickeritus).
Thanks,
Segher
next prev parent reply other threads:[~2016-08-27 1:24 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-04 4:34 [PATCH], Improve vector int/long initialization on PowerPC Michael Meissner
2016-08-04 15:03 ` Segher Boessenkool
2016-08-08 22:55 ` Michael Meissner
2016-08-11 23:15 ` [PATCH], Patch #4, " Michael Meissner
2016-08-12 0:21 ` Segher Boessenkool
2016-08-19 22:18 ` [PATCH], Patch #5, Improve vector int " Michael Meissner
2016-08-19 23:59 ` [PATCH], Patch #6, Improve vector short/char splat " Michael Meissner
2016-08-22 16:47 ` Segher Boessenkool
2016-08-26 19:30 ` [PATCH], Patch #7, Add PowerPC vector initialization tests Michael Meissner
2016-08-27 1:24 ` Segher Boessenkool [this message]
2016-08-29 19:35 ` Michael Meissner
2016-08-22 16:38 ` [PATCH], Patch #5, Improve vector int initialization on PowerPC Segher Boessenkool
2016-08-22 22:01 ` Michael Meissner
2016-08-22 22:57 ` Segher Boessenkool
2016-08-05 22:00 ` [PATCH], Improve vector int/long " Pat Haugen
2016-08-08 22:56 ` Michael Meissner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160827012439.GA20847@gate.crashing.org \
--to=segher@kernel.crashing.org \
--cc=dje.gcc@gmail.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=meissner@linux.vnet.ibm.com \
--cc=wschmidt@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).