Re: [Xenomai] [Xenomai-git] Jan Kiszka : switchtest: Add SSE and AVX check

[Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>]

On 04/16/2013 09:34 AM, Jan Kiszka wrote:

> On 2013-04-16 09:32, Gilles Chanteperdrix wrote:
>> On 04/16/2013 09:24 AM, Jan Kiszka wrote:
>>
>>> On 2013-04-16 00:02, Jan Kiszka wrote:
>>>> On 2013-04-13 18:11, Gilles Chanteperdrix wrote:
>>>>> This routine causes a warning from gcc and looks indeed wrong: if the
>>>>> "vec" variable is used as an output variable of the inline assembly, it
>>>>> should be in the output section of the inline assembly, not the input
>>>>> section.
>>>>>
>>>>
>>>> Yes, seems wrong. Will try to look into it the next days.
>>>
>>> Done, you can find the obvious fix in my for-upstream queue.
>>
>>
>> Hi Jan,
>>
>> yes the fix is obvious, however the fact that machines with AVX support
>> still pass xeno-regression-test is not, have you checked that?
> 
> I've checked that the binary output is unaffected, at least for my
> compiler here. Makes sense as there is no other use for the registers
> around, and the assembler just blindly put the input variables into the
> destination slot of the instructions.


No, it does not really make sense: a piece of assembly without any side
effects and any outputs is basically useless, so the compiler could very
well decide to optimize it out. But if you have checked, OK.


-- 
                                                                Gilles.