[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] Add `fixnum?' VM primitive
From: |
Andy Wingo |
Subject: |
Re: [PATCH 3/3] Add `fixnum?' VM primitive |
Date: |
Mon, 04 Apr 2011 23:53:51 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Hi Andreas,
I applied the first two, thanks.
I am hesitant about this one for three reasons:
1) I don't want the compiler to import (rnrs arithmetic fixnums).
Rather, if we were to do this, I would have that module register its
primitives, as GOOPS does.
2) Something about this sounds wrong to me. If fixnum? is important,
why not have it in Guile's default environment directly? Or some
other non-R6RS namespace. What about, for example,
`exact-integer?'? And why for fixnums and not flonums?
3) Are there no alternatives, like exposing Guile's tags to Scheme
directly? (ice-9 tags) for example. Then we could inline tc3?,
tc8?, and tc16? ops. Some other predicates would become
unnecessary; char? for example. Perhaps this is a bad idea though.
Apologies for being crabby here after you've done all this work :) It
could well be that something close to this is the right thing.
Cheers,
Andy
--
http://wingolog.org/
- R6RS fixnum arithmetic optimizations, Andreas Rottmann, 2011/04/02
- [PATCH 1/3] Add a few benchmarks for R6RS fixnum arithmetic, Andreas Rottmann, 2011/04/02
- [PATCH 2/3] Several optimizations for R6RS fixnum arithmetic, Andreas Rottmann, 2011/04/02
- [PATCH 3/3] Add `fixnum?' VM primitive, Andreas Rottmann, 2011/04/02
- Re: [PATCH 3/3] Add `fixnum?' VM primitive,
Andy Wingo <=
- Re: [PATCH 3/3] Add `fixnum?' VM primitive, Andreas Rottmann, 2011/04/04
- define-inlinable, Ludovic Courtès, 2011/04/06
- Re: define-inlinable, Andreas Rottmann, 2011/04/06
- Re: define-inlinable, Ludovic Courtès, 2011/04/06
- Re: define-inlinable, Andy Wingo, 2011/04/11
- Re: define-inlinable, Ludovic Courtès, 2011/04/11
- Re: define-inlinable, Andy Wingo, 2011/04/11
- Re: define-inlinable, Andreas Rottmann, 2011/04/11
- Re: [PATCH 3/3] Add `fixnum?' VM primitive, Ludovic Courtès, 2011/04/07