Sharp eye! Thanks. Updated patch is attached. Guys, with write approval, could you please commit that?
Thans, K
On Wed, Jul 27, 2011 at 8:46 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
> 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.
>
bmi1-3.testcases.gcc.patch
Description: Binary data
