Bug#479529: Upstream fix is partially incorrect

2008-07-07 Thread Christian Thalinger
On Mon, 2008-07-07 at 17:17 +0100, Thiemo Seufer wrote: > Christian Thalinger wrote: > > On Mon, 2008-07-07 at 12:49 +0100, Thiemo Seufer wrote: > > > I leave the Debian bug open for this one. > > > > Is there still a problem somewhere? Did I miss something? > > I believe the SVN commit you poin

Bug#479529: Upstream fix is partially incorrect

2008-07-07 Thread Thiemo Seufer
Christian Thalinger wrote: > On Mon, 2008-07-07 at 12:49 +0100, Thiemo Seufer wrote: > > I leave the Debian bug open for this one. > > Is there still a problem somewhere? Did I miss something? I believe the SVN commit you pointed at is not yet in the Debian package. Thiemo -- To UNSUBSCRIB

Bug#479529: Upstream fix is partially incorrect

2008-07-07 Thread Christian Thalinger
On Mon, 2008-07-07 at 12:49 +0100, Thiemo Seufer wrote: > I leave the Debian bug open for this one. Is there still a problem somewhere? Did I miss something? - twisti -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#479529: Upstream fix is partially incorrect

2008-07-07 Thread Thiemo Seufer
Christian Thalinger wrote: > I guess you are talking about this code: > > http://mips.complang.tuwien.ac.at/hg/cacao/file/2effc42cfb01/src/vm/jit/mips/asmpart.S#l426 > > This code is correct, as the fsX defines use paired registers, starting > with a even one. Also see: > > http://mips.complang

Bug#479529: Upstream fix is partially incorrect

2008-07-07 Thread Christian Thalinger
I guess you are talking about this code: http://mips.complang.tuwien.ac.at/hg/cacao/file/2effc42cfb01/src/vm/jit/mips/asmpart.S#l426 This code is correct, as the fsX defines use paired registers, starting with a even one. Also see: http://mips.complang.tuwien.ac.at/hg/cacao/file/2effc42cfb01/sr

Bug#479529: Upstream fix is partially incorrect

2008-05-14 Thread Thiemo Seufer
reopen 479529 thanks Upstream's version of the patch is incorrect for the SIZOF_VOID_P != 8 case. For the O32 ABI, l.d/s.d expand to a instruction sequence which handles the even/odd FPU register pairs, so the programmer can just think of it as "load double / store double" without ever explicitly