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