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@comcast.net>
To: "Jean-Marc Valin" <Jean-Marc.Valin@USherbrooke.ca>
Cc: <speex-dev@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
