[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#384243: FTBFS (alpha): assembly syntax errors



Below is a better fix for the s4addq error (saves one instruction) :

---------------------------------------------------------------------------------------
quercia@salon:~/src/caml/numerix$ diff -Nurp numerix-0.22/kernel/n/alpha/toom.S numerix-0.22a/kernel/n/alpha/toom.S
--- numerix-0.22/kernel/n/alpha/toom.S  2006-01-12 14:25:42.000000000 +0100
+++ numerix-0.22a/kernel/n/alpha/toom.S 2006-08-23 07:36:31.000000000 +0200
@@ -372,11 +372,11 @@ L(nogp):

       ldq    $16, _c_
       ldq    $17, _p_
+       s4addq $17,  4,  $0     # r0 <- 4p+4
       addq   $17, 3,   $17    # r17 <- p+3
       s8addq $17, $16, $18    # r18 <- &c[p+3]
       bis    $17, $17, $19    # r19 <- p+3
       lda    $20, _d_
-       s4addq $17, -8,  $0     # r0 <- 4p+4
       s8addq $0,  $20, $20    # r20 <- &f

# diminue les longeurs des facteurs si les chiffres de tête sont nuls
@@ -956,9 +956,9 @@ L(nogp):
ldq $16, _c_
       ldq    $17, _p_
+       s4addq $17, 4,   $0     # r0 <- 4p+4
       addq   $17, 3,   $17    # r17 <- p+3
       lda    $18, _d_
-       s4addq $17, -8,  $0     # r0 <- 4p+4
       s8addq $0,  $18, $18    # r18 <- &f

# diminue les longeurs des facteurs si les chiffres de tête sont nuls
---------------------------------------------------------------------------------------

This patch, along with the missing commas fixes is included in unofficial numerix-0.22a (http://pauillac.inria.fr/~quercia/cdrom/bibs/numerix-0.22a.tar.gz).
I've tested it on the Digital computer, everything seems OK.

Regards,

--
Michel Quercia
23 rue de Montchapet, 21000 Dijon
http://michel.quercia.free.fr (maths)
http://pauillac.inria.fr/~quercia (informatique)
mailto:michel.quercia@prepas.org




Reply to: