GCC Maintainers:
The following patch fixes the instruction count tests for various
builtin testcases that fail on AIX. The expected instruction counts
differ in a few cases between Linux and AIX. An AIX instruction target
was added to the instruction count tests that differ on AIX and Linux.
Add
Hi!
On Thu, Jun 14, 2018 at 08:12:31AM -0700, Carl Love wrote:
> --- a/gcc/testsuite/gcc.target/powerpc/altivec-7.c
> +++ b/gcc/testsuite/gcc.target/powerpc/altivec-7.c
> @@ -85,17 +85,22 @@ int main ()
> /* { dg-final { scan-assembler-times "vpkpx" 2 } } */
> /* { dg-final { scan-assembler-time
Segher:
Per our discussions, the previous patch had issues with the target
!powerpc*-*-aix* not working correctly and thus the instruction count
test was not being done. I have addressed those issues and verified by
inspecting the gcc/testsuite/gcc/gcc.log file to make sure the test was
actually
Hi Carl,
On Mon, Jun 18, 2018 at 01:52:15PM -0700, Carl Love wrote:
> Per our discussions, the previous patch had issues with the target
> !powerpc*-*-aix* not working correctly and thus the instruction count
> test was not being done. I have addressed those issues and verified by
> inspecting th
Segher:
I believe I have addressed all of your concerns with the patch.
I have retested it and it looks good.
Please let me know if the patch looks OK for GCC mainline.
Carl Love
>From 8d354f93c5ddb5161b
Hi Carl,
On Wed, Jun 20, 2018 at 05:09:00PM -0700, Carl Love wrote:
> I believe I have addressed all of your concerns with the patch.
>
> I have retested it and it looks good.
It looks good indeed. Please commit, thanks!
I noticed one more thing (follow-up patch?)
> /* { dg-final { scan-asse