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

[Jim Crichton]

>> 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