Ok, after class I will fix and repatch. Making opcode_t a simple type that
is configurable.

Thanks!
Tanton

-----Original Message-----
From: Dan Sugalski
To: Simon Cozens; ''[EMAIL PROTECTED]' '
Sent: 9/19/2001 10:14 AM
Subject: Re: [PATCH] changing IV to opcode_t!!

At 03:58 PM 9/19/2001 +0100, Simon Cozens wrote:
>On Wed, Sep 19, 2001 at 10:59:45AM -0400, Dan Sugalski wrote:
> > Nope. opcode_t should be the native opcode type for the platform
we're
> > compiling on. No need for fancy unions--configure should find out
the
> > integer type that works out right for the platform and the bytecode
and 
> use
> > that.
>
>I thought the whole point was that on some platforms there *isn't* an
>integer type that works right.

Nope. There's *always* an integer type that works right. We won't be 
running on a platform that doesn't have 32-bit integers. (Well, not
without 
someone hacking the heck out of the core)

The bytecode stream on disk is a series of M-bit integers holding 32-bit

opcodes and parameters. It's treated as a stream of N-bit integers 
internally. That doesn't mean that M==32, or N==32, or even M==N.

If M != N it means the bytecode load function needs to read in the
bytecode 
stream from disk, treat it as a series of M-bit integers that need to be

converted to N-bit ones for native interpretation.


                                        Dan

--------------------------------------"it's like
this"-------------------
Dan Sugalski                          even samurai
[EMAIL PROTECTED]                         have teddy bears and even
                                      teddy bears get drunk

Reply via email to