On Wed, Jul 27, 2011 at 5:02 PM, Kirill Yukhin <kirill.yuk...@gmail.com> wrote:

> Thanks, for inputs.
> Sure, lzcnt useless here. I am updated and tested BMI detection in test 
> driver.
>
> testuite/ChageLog entry:
> 2011-07-27  Yukhin Kirill  <kirill.yuk...@intel.com>
>
>        * gcc.target/i386/i386.exp (check_effective_target_bmi): New.
>        * gcc.target/i386/bmi-andn-1.c: New test.
>        * gcc.target/i386/bmi-andn-1a.c: Likewise.
>        * gcc.target/i386/bmi-andn-2.c: Likewise.
>        * gcc.target/i386/bmi-andn-2a.c: Likewise.
>        * gcc.target/i386/bmi-bextr-1.c: Likewise.
>        * gcc.target/i386/bmi-bextr-1a.c: Likewise.
>        * gcc.target/i386/bmi-bextr-2.c: Likewise.
>        * gcc.target/i386/bmi-bextr-2a.c: Likewise.
>        * gcc.target/i386/bmi-blsi-1.c: Likewise.
>        * gcc.target/i386/bmi-blsi-1a.c: Likewise.
>        * gcc.target/i386/bmi-blsi-2.c: Likewise.
>        * gcc.target/i386/bmi-blsi-2a.c: Likewise.
>        * gcc.target/i386/bmi-blsmsk-1.c: Likewise.
>        * gcc.target/i386/bmi-blsmsk-1a.c: Likewise.
>        * gcc.target/i386/bmi-blsmsk-2.c: Likewise.
>        * gcc.target/i386/bmi-blsmsk-2a.c: Likewise.
>        * gcc.target/i386/bmi-blsr-1.c: Likewise.
>        * gcc.target/i386/bmi-blsr-1a.c: Likewise.
>        * gcc.target/i386/bmi-blsr-2.c: Likewise.
>        * gcc.target/i386/bmi-blsr-2a.c: Likewise.
>        * gcc.target/i386/bmi-tzcnt-1.c: Likewise.
>        * gcc.target/i386/bmi-tzcnt-1a.c: Likewise.
>        * gcc.target/i386/bmi-tzcnt-2.c: Likewise.
>        * gcc.target/i386/bmi-tzcnt-2a.c: Likewise.
>
>
> New patch is attached.
> Is it OK?

+++ b/gcc/testsuite/gcc.target/i386/bmi-tzcnt-1a.c
@@ -0,0 +1,6 @@
+/* { dg-do compile { target { ! ia32 } } } */
+/* { dg-options "-O2 -mbmi -fno-inline -dp" } */
+
+#include "bmi-tzcnt-1.c"
+
+/* { dg-final { scan-assembler-times "tzcntq" 1 } } */

You don't need -dp there.

The patch is OK for mainline with this change.

Thanks,
Uros.

Reply via email to