[M3devel] M3CG

[Daniel Alejandro Benavides D.]

Hi all:
strictly speaking isn't allowed target typing if that's what you are referring to.
Thanks in advance

--- El mar, 4/9/12, Jay <jay.krell at cornell.edu> escribió:

De: Jay <jay.krell at cornell.edu>
Asunto: Re: [M3devel] M3CG
Para: "Antony Hosking" <hosking at cs.purdue.edu>
CC: "m3devel at elegosoft.com developers" <m3devel at elegosoft.com>, "Jay K" <jay.krell at cornell.edu>
Fecha: martes, 4 de septiembre, 2012 11:09

"BITS" seems to not provide any useful value. It only makes it so you can't put the type into a portable unpacked record, which is what I was doing. I either have to pack my record somehow (counting target-dependent bits??) or lay out the fields carefully in some way that all targets allow. The problem was that some targets didn't like the layout.



Given the stated range, that you'll get enough bits to store it, what is the point in giving the size too? Isn't providing just the range and no particular size, sufficient & more abstract? Granted, I went the other way.



I don't remember but guessing:


Cardinal.T would be, like CARDINAL vs. INTEGER: only slightly subtley useful: same as TInt.T, but disallows negative numbers.


Most likely I ran into places where a host integer/cardinal did not necessarily suffice, and so a target type was called for.


I know we have places that need to  use target types but don't yet. Specifically for the sizes of things like records & arrays. Otherwise we have hacks where 64bit systems declare 32bit limits.


 - Jay (briefly/pocket-sized-computer-aka-phone)

On Sep 4, 2012, at 7:11 AM, Antony Hosking <hosking at cs.purdue.edu> wrote:

> Jay,
> 
> I’ve been looking over some of your changes to M3CG interfaces.  You’ll notice that I removed your import of Cstdlib.int into M3CG_Ops.i3.  It does not belong there.  The type as declared:
> 
> TypeUID = BITS 32 FOR [-16_7fffffff-1 .. 16_7fffffff];
> 
> is correct as is.  It is a 32 bit value that encodes a particular subrange, REGARDLESS of target machine.  It is improper to change that definition to rely on some C type.
> 
> I also have some other questions.  Why did you add the type Cardinal.T?  This seems entirely unnecessary, since targets don’t have a Cardinal type.  The only type they have is a target integer.  All other Modula-3 ordinal types should be simulated in the compiler using TInt.T.  I would advise that it be removed.  Also, I don’t understand why you changed descriptions of the primitive types to use this Cardinal.T instead of the original CARDINAL to hold information about the bit size, alignment, and byte size.  There is no situation in which a host will need to emulate the behavior of CARDINAL for these values (all can be represented using CARDINAL no matter what the host).
> 
> I am concerned that these changes reflect a desire on your part to change the world to fit your specific needs (whether justified or not).  The interfaces defined in M3CG were carefully designed and inherit from a long code-chain going back to the 1980’s and have not see huge changes since then.  I strongly advise against making changes unless you have good reason, because there are a number of tools that rely on M3CG (not just those in the public sources).
> 
> I am going to do a pass to revert some of your changes, since they are causing a number of my systems to fail to build.  (cf. Niagara on tinderbox).
> 
> I strongly advise that you try to use a private branch when developing new functionality.  We as a community can then go through a revision process to vet substantive changes before merging them into the trunk.
> 
> — Tony
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://m3lists.elegosoft.com/pipermail/m3devel/attachments/20120904/9cb89435/attachment-0002.html>