CVS commit: src/sys/arch/m68k/fpe
Module Name:src Committed By: isaki Date: Sun Nov 19 03:58:15 UTC 2023 Modified Files: src/sys/arch/m68k/fpe: fpu_rem.c Log Message: m68k: Remove an unused variable since rev 1.1. Detected by clang15 (nono emulator has imported and used this FPE). To generate a diff of this commit: cvs rdiff -u -r1.17 -r1.18 src/sys/arch/m68k/fpe/fpu_rem.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
CVS commit: src/sys/arch/m68k/fpe
Module Name:src Committed By: isaki Date: Sun Nov 19 03:58:15 UTC 2023 Modified Files: src/sys/arch/m68k/fpe: fpu_rem.c Log Message: m68k: Remove an unused variable since rev 1.1. Detected by clang15 (nono emulator has imported and used this FPE). To generate a diff of this commit: cvs rdiff -u -r1.17 -r1.18 src/sys/arch/m68k/fpe/fpu_rem.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. Modified files: Index: src/sys/arch/m68k/fpe/fpu_rem.c diff -u src/sys/arch/m68k/fpe/fpu_rem.c:1.17 src/sys/arch/m68k/fpe/fpu_rem.c:1.18 --- src/sys/arch/m68k/fpe/fpu_rem.c:1.17 Thu Feb 5 12:22:06 2015 +++ src/sys/arch/m68k/fpe/fpu_rem.c Sun Nov 19 03:58:15 2023 @@ -1,4 +1,4 @@ -/* $NetBSD: fpu_rem.c,v 1.17 2015/02/05 12:22:06 isaki Exp $ */ +/* $NetBSD: fpu_rem.c,v 1.18 2023/11/19 03:58:15 isaki Exp $ */ /* * Copyright (c) 1995 Ken Nakata @@ -32,7 +32,7 @@ */ #include -__KERNEL_RCSID(0, "$NetBSD: fpu_rem.c,v 1.17 2015/02/05 12:22:06 isaki Exp $"); +__KERNEL_RCSID(0, "$NetBSD: fpu_rem.c,v 1.18 2023/11/19 03:58:15 isaki Exp $"); #include #include @@ -48,7 +48,7 @@ __KERNEL_RCSID(0, "$NetBSD: fpu_rem.c,v *signQ := signX EOR signY. Record whether MOD or REM *is requested. * - * Step 2. Set L := expo(X)-expo(Y), k := 0, Q := 0. + * Step 2. Set L := expo(X)-expo(Y), Q := 0. *If (L < 0) then * R := X, go to Step 4. *else @@ -59,8 +59,7 @@ __KERNEL_RCSID(0, "$NetBSD: fpu_rem.c,v *3.1 If R = Y, then { Q := Q + 1, R := 0, go to Step 7. } *3.2 If R > Y, then { R := R - Y, Q := Q + 1} *3.3 If j = 0, go to Step 4. - *3.4 k := k + 1, j := j - 1, Q := 2Q, R := 2R. Go to - *Step 3.1. + *3.4 j := j - 1, Q := 2Q, R := 2R. Go to Step 3.1. * * Step 4. R := signX*R. * @@ -105,7 +104,7 @@ __fpu_modrem(struct fpemu *fe, int is_mo static struct fpn X, Y; struct fpn *x, *y, *r; uint32_t signX, signY, signQ; - int j, k, l, q; + int j, l, q; int cmp; if (ISNAN(&fe->fe_f1) || ISNAN(&fe->fe_f2)) @@ -138,7 +137,6 @@ __fpu_modrem(struct fpemu *fe, int is_mo * Step 2 */ l = x->fp_exp - y->fp_exp; - k = 0; CPYFPN(r, x); if (l >= 0) { r->fp_exp -= l; @@ -168,7 +166,6 @@ __fpu_modrem(struct fpemu *fe, int is_mo goto Step4; /* Step 3.4 */ - k++; j--; q += q; r->fp_exp++;
CVS commit: src/sys/arch/m68k/fpe
Module Name:src Committed By: andvar Date: Sun Sep 17 13:14:08 UTC 2023 Modified Files: src/sys/arch/m68k/fpe: fpu_emulate.c Log Message: word was changed to sval at rev 1.39, but DPRINTF values were not updated. Fixes DEBUG_FPE enabled build for 68k FPE code. To generate a diff of this commit: cvs rdiff -u -r1.40 -r1.41 src/sys/arch/m68k/fpe/fpu_emulate.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. Modified files: Index: src/sys/arch/m68k/fpe/fpu_emulate.c diff -u src/sys/arch/m68k/fpe/fpu_emulate.c:1.40 src/sys/arch/m68k/fpe/fpu_emulate.c:1.41 --- src/sys/arch/m68k/fpe/fpu_emulate.c:1.40 Fri Dec 27 07:41:23 2019 +++ src/sys/arch/m68k/fpe/fpu_emulate.c Sun Sep 17 13:14:08 2023 @@ -1,4 +1,4 @@ -/* $NetBSD: fpu_emulate.c,v 1.40 2019/12/27 07:41:23 msaitoh Exp $ */ +/* $NetBSD: fpu_emulate.c,v 1.41 2023/09/17 13:14:08 andvar Exp $ */ /* * Copyright (c) 1995 Gordon W. Ross @@ -37,7 +37,7 @@ */ #include -__KERNEL_RCSID(0, "$NetBSD: fpu_emulate.c,v 1.40 2019/12/27 07:41:23 msaitoh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: fpu_emulate.c,v 1.41 2023/09/17 13:14:08 andvar Exp $"); #include #include @@ -132,12 +132,12 @@ fpu_emulate(struct frame *frame, struct if ((sval & 0xf000) != 0xf000) { DPRINTF(("%s: not coproc. insn.: opcode=0x%x\n", - __func__, word)); + __func__, sval)); fpe_abort(frame, ksi, SIGILL, ILL_ILLOPC); } if ((sval & 0x0E00) != 0x0200) { - DPRINTF(("%s: bad coproc. id: opcode=0x%x\n", __func__, word)); + DPRINTF(("%s: bad coproc. id: opcode=0x%x\n", __func__, sval)); fpe_abort(frame, ksi, SIGILL, ILL_ILLOPC); }
CVS commit: src/sys/arch/m68k/fpe
Module Name:src Committed By: andvar Date: Sun Sep 17 13:14:08 UTC 2023 Modified Files: src/sys/arch/m68k/fpe: fpu_emulate.c Log Message: word was changed to sval at rev 1.39, but DPRINTF values were not updated. Fixes DEBUG_FPE enabled build for 68k FPE code. To generate a diff of this commit: cvs rdiff -u -r1.40 -r1.41 src/sys/arch/m68k/fpe/fpu_emulate.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Re: CVS commit: src/sys/arch/m68k/fpe
At Fri, 19 Apr 2013 18:18:57 + (UTC), Christos Zoulas wrote: > >Module Name: src > >Committed By:isaki > >Date:Fri Apr 19 13:31:11 UTC 2013 > > > >Modified Files: > > src/sys/arch/m68k/fpe: files.fpe fpu_emulate.h fpu_hyperb.c fpu_trig.c > >Added Files: > > src/sys/arch/m68k/fpe: fpu_cordic.c > > Thanks! > > s/tayler/taylor/g Oops, thanks! --- Tetsuya Isaki
Re: CVS commit: src/sys/arch/m68k/fpe
In article <20130419133111.845e917...@cvs.netbsd.org>, Tetsuya Isaki wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: isaki >Date: Fri Apr 19 13:31:11 UTC 2013 > >Modified Files: > src/sys/arch/m68k/fpe: files.fpe fpu_emulate.h fpu_hyperb.c fpu_trig.c >Added Files: > src/sys/arch/m68k/fpe: fpu_cordic.c Thanks! s/tayler/taylor/g christos