[gmx-developers] Re: CVS nb_kernel204_ia32_sse.s as problem

[Berk Hess]

Rafael R. Pappalardo wrote:

>>Hi,
>>
>>A colleague of mine just tried to compile the current CVS code
>>on his laptop with SUSE 9.3, but as gives an error:
>>
>>`/home/sayar/gmx/src/gmxlib/nonbonded/nb_kernel_ia32_sse'
>>/bin/sh ../../../../libtool --mode=compile cc  -O3 -fomit-frame-pointer 
>>    
>>
>-finline-functions -Wall -Wno-unused -malign-double -funroll-all-loops -c -o 
>nb_kernel204_ia32_sse.lo nb_kernel204_ia32_sse.s
>  
>
>> cc -O3 -fomit-frame-pointer -finline-functions -Wall -Wno-unused 
>>    
>>
>-malign-double -funroll-all-loops -c nb_kernel204_ia32_sse.s -o 
>nb_kernel204_ia32_sse.o
>  
>
>>nb_kernel204_ia32_sse.s: Assembler messages:
>>nb_kernel204_ia32_sse.s:1696: Error: invalid operand for 'mov' ('push' 
>>    
>>
>unexpected)
>  
>
>>make[5]: *** [nb_kernel204_ia32_sse.lo] Error 1
>>
>>
>>gcc (GCC) 3.3.5 20050117 (prerelease) (SUSE Linux)
>>
>>GNU assembler 2.15.94.0.2.2 20041220 (SuSE Linux)
>>
>>Berk.
>>    
>>
>
>I belive that the error is a typo. If you edit the file and press return 
>before the push in line 1696 the error goes away.
>  
>
Thank you very much for this tip!
I didn't dare to look into the assembly myself, but it is a very simple 
error.

I just hope that all the older versions of as that did not complain
interpreted this line correctly.

I have committed the fix in CVS.

Berk.