This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix implementation of vec_packsu (vector unsigned long long, vector unsigned long long) built-in function
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Kelvin Nilsen <kdnilsen at linux dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 19 Jun 2018 14:30:31 -0500
- Subject: Re: [PATCH, rs6000] Fix implementation of vec_packsu (vector unsigned long long, vector unsigned long long) built-in function
- References: <98281c7a-a9d8-9827-7804-a6c8a9a650fb@linux.ibm.com>
Hi!
On Mon, Jun 18, 2018 at 11:29:55AM -0500, Kelvin Nilsen wrote:
> +/* A single vpkudus matches twice because this is compiled with -dp,
> + causing diagnostic comments to appear in the resulting .s file, one
> + of which matches vpkudus. */
-dp prints the name of the instruction pattern, which is altivec_vpkudus.
So if you look for the full word instead, this problem isn't there I
think?
> +/* { dg-final { scan-assembler-times "vpkudus" 2 } } */
/* { dg-final { scan-assembler-times {\mvpkudus\M} 1 } } */
Okay with that change (and comment changes). Thanks!
Segher