Re: [Lame-dev] RE: LAME 3.94 beta suggestions

[Gabriel B. <bou...@mp...>]

Ivan Farkas wrote:

> I made some modifications to the INSTALL and Makefile.MSVC files.
> This made it possible to compile using Microsoft Visual Studio 2003 (MSVC7)
> taking advantage of the Pentium 4 and AMD Athlon optimization.
> CPU=P4 switch works only for VS 2003.

Sorry, breaking compatibility with VC6 is not a good option.

> I also wrote a small Build_MSVC7.cmd file that contains this command below:
> 
> nmake -f Makefile.MSVC comp=msvc asm=no CPU=P4 /A

Do you know that you can directly ask VC to compile something in a dsw? 
It makes the makefile almost unnecessary.
Moreover, you are using P4 compiler optimisations, but not using asm 
optimisations from Lame. You will most likely end up with a slower 
binary than a simple compile using our current options but using 
assembly optimisations.


Perhaps your changes are usefull to you, but I think that this is not 
our case.

Regards,

-- 
Gabriel Bouvigne
www.mp3-tech.org
personal page: http://gabriel.mp3-tech.org