--- John Steele Scott <[EMAIL PROTECTED]> wrote: > Josh Coalson <[EMAIL PROTECTED]> writes: > > > --- John Steele Scott <[EMAIL PROTECTED]> wrote: > >> Back in October 2004, I did a bit of work on FLAC to get version > >> 1.1.1 to > >> build correctly under GNU/Linux/PPC. Only now have I realised that > >> somewhere > >> along the way something broke in FLAC's decoding. On my machine, > >> roughly 50% > >> of FLAC files are being decoded incorrectly. > >> > >> I presume that I broke the code somehow when translating Josh's > >> original OS X > >> assembler into GNU style. > > > > are you the one who submitted the patch with a translated > > version of lpc_asm.s? I thought that was Luca. > > I'm not sure exactly what happened. What you have in CVS: > <http://cvs.sourceforge.net/viewcvs.py/flac/flac/src/libFLAC/ppc/gas/lpc_asm.s?rev=1.1&view=log> > > is not the same as the patch applied by Gentoo: > <http://www.gentoo.org/cgi-bin/viewcvs.cgi/media-libs/flac/files/flac-1.1.1-altivec.patch.gz> > > There is little difference between either version except for the > comment > style, and the way constant addresses are specified. Also the Gentoo > patch > patches cpu.c, otherwise the Altivec code path isn't taken.
the one I checked I thought I got directly from Luca on this list. but I have lost track... > I'm trying to get a handle on where I should be looking to fix the > problem. If > your saying that the gas version in FLAC CVS has been validated, the > easiest > solution is to just use that one. Otherwise I need to go back to the > Apple > version. Presumably that one has been thouroughly checked? no, not yet, I haven't got to the point of straightening anything out totally. I know this is bad, but can I ask you guys (John/Luca/Chris) to take a look at the latest CVS and try to get this working? the PPC improvements are very important for macs and Chris has a lot of optimizations that have been backing up since I can't sort things out. the main problem is that the asm syntax for 'as' (the one from apple that ships with darwin) and 'gas' are apparently different. it sounds like for ppc-linux to work, there has to be a 'gas' version. all I have is an ibook that I don't want to mess with too much so for me to do any work on it there has to be an 'as' version as well. since I don't know how to resolve this, in current CVS I have checked in a system where there are src/libFLAC/ppc/as and src/libFLAC/ppc/gas selected by configure. I have also checked in as many patches as I could make sense of (the cpu.c detection stuff and the configure.in stuff). can you guys take a look at current CVS and help me get something that will build both ways? after that it will be much easier for me to start looking at correctness. thanks, Josh __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com _______________________________________________ Flac-dev mailing list Flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
