Re: [PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-27 Thread Andreas Rottmann
l...@gnu.org (Ludovic Courtès) writes: > Hello, > > Andreas Rottmann writes: > >> So, that's around 5% improvment (on the ZIP benchmark) for an IMHO >> significantly more hackish implementation. I'm not sure that's worth >> it. WDYT? > > Was it with ‘fixnum?’ inline, or with the ‘fixnum?’ instru

Re: [PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-27 Thread Ludovic Courtès
Hello, Andreas Rottmann writes: > So, that's around 5% improvment (on the ZIP benchmark) for an IMHO > significantly more hackish implementation. I'm not sure that's worth > it. WDYT? Was it with ‘fixnum?’ inline, or with the ‘fixnum?’ instruction? It’s ironic that while R6RS fixnums are a pe

Re: [PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-25 Thread Andreas Rottmann
Andreas Rottmann writes: > l...@gnu.org (Ludovic Courtès) writes: > >> Hi Andreas, >> >> I’m all for your suggestion. >> >> Andreas Rottmann writes: >> >>> + (define-syntax define-fxop* >>> +(syntax-rules () >>> + ((_ name op) >>> + (define name >>> +(case-lambda >>> + (

Re: [PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-24 Thread Andreas Rottmann
l...@gnu.org (Ludovic Courtès) writes: > Hi Andreas, > > I’m all for your suggestion. > > Andreas Rottmann writes: > >> + (define-syntax define-fxop* >> +(syntax-rules () >> + ((_ name op) >> + (define name >> + (case-lambda >> + ((x y) >> +(assert-fixnum x y) >>

Re: [PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-24 Thread Ludovic Courtès
Hi Andreas, I’m all for your suggestion. Andreas Rottmann writes: > + (define-syntax define-fxop* > +(syntax-rules () > + ((_ name op) > + (define name > + (case-lambda > +((x y) > + (assert-fixnum x y) > + (op x y)) > +(args > + (ass

[PATCH] Take some lowhanging fruit to speed up R6RS fixnum operations

2011-03-22 Thread Andreas Rottmann
* module/rnrs/arithmetic/fixnums.scm (assert-fixnum): Is now a macro. (assert-fixnums): New procedure checking a the elements of a list for fixnum-ness. All callers applying `assert-fixnum' to a list now changed to use this procedure. * module/rnrs/arithmetic/fixnums.scm (define-fxop*): N