Re: [PATCH, testsuite, i386] FMA3 testcases + typo fix in MD

[Kirill Yukhin <kirill dot yukhin at gmail dot com>]

Wow, it works!

Thank you. New patch attached.
ChangeLogs were not touched.

Tests pass both on ia32/x86-64 with and without simulator.

Thanks, K

On Thu, Oct 6, 2011 at 2:07 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Thu, Oct 6, 2011 at 11:38 AM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
>
>> Thanks for review. I did all but one
>>
>>> you have disabled all tests on ia32 - unconditionally use "-O2 -mfma
>>> -mfpmath=sse" for dg-options, and these instructions will magically
>>> appear on all targets.
>>
>> I am enabling these tests to run on ia32, they all fail to scan-assembler,
>> since assemler is completely different, here is instance:
>> test_neg_sub_neg_sub:
>> .LFB15:
>> ? ? ? ?.cfi_startproc
>> ? ? ? ?subl ? ?$12, %esp
>> ? ? ? ?.cfi_def_cfa_offset 16
>> ? ? ? ?vmovsd ?16(%esp), %xmm2
>> ? ? ? ?vmovsd ?24(%esp), %xmm1
>> ? ? ? ?vmovapd %xmm2, %xmm0
>> ? ? ? ?vfnmsub213sd ? ?32(%esp), %xmm1, %xmm0
>> ? ? ? ?vfnmsub132sd ? ?%xmm2, %xmm1, %xmm0
>> ? ? ? ?vmovsd ?%xmm0, (%esp)
>> ? ? ? ?fldl ? ?(%esp)
>> ? ? ? ?addl ? ?$12, %esp
>> ? ? ? ?.cfi_def_cfa_offset 4
>> ? ? ? ?ret
>> ? ? ? ?.cfi_endproc
>>
>> On ia32 params are passed completely different and therefore code differs.
>
> You can add __attribute__((sseregparm)) to function declaration. This
> will force arguments to/from function to SSE registers. The problem
> is, that it will result in "warning: 'sseregparm' attribute ignored",
> but this can be ignored using dg-prune-output dg directive. Please see
> many examples in the testsuite.
>
> Uros.
>

Attachment: fma3_typo_tests-3.gcc.patch
Description: Binary data