[Speex-dev] Speex echo canceller on TI C55 DSP

[Jim Crichton]

The builds that I referred to in my last mail should be 11398 and 11387, not 
11387 and 11343.  Sorry for any confusion.

- Jim

----- Original Message ----- 
From: "Jim Crichton" <jim.crichton at comcast.net>
To: "Jean-Marc Valin" <Jean-Marc.Valin at USherbrooke.ca>
Cc: <speex-dev at xiph.org>
Sent: Wednesday, May 10, 2006 11:47 AM
Subject: Re: [Speex-dev] Speex echo canceller on TI C55 DSP


>>> My builds for the two platforms used exactly the same source files, 
>>> though
>>> there were a few ifdefs in the test_echo routine to deal with file I/O 
>>> for
>>> the C55 with its 16-bit char size.  When I get a chance, I will add some
>>> instrumentation and see if I can find where things diverge.  Just 
>>> looking at
>>> the canceled audio files with a waveform editor, they were not obviously
>>> different, so it is not a burning issue for me.
>>
>> Try updating svn again. I just found out that EXTEND32() and EXTRACT16()
>> were actually unsafe due to a pair of () missing around the argument. I
>> also fixed a few minor things. Tell me if it works now.
>
> The C55 and C64 builds diverge in exactly the same place as before (byte 
> 0x1000).  The output from C55 build 11387 (svn head) diverges from C55 
> build 11343 slightly later (byte 0x1116).  Similarly, the output from C64 
> build 11387 (svn head) diverges from C64 build 11343 slightly later (byte 
> 0x1126). So, your change clearly had an effect, not just on the 16-bit 
> machine, but there is something else going on.
>
> - Jim