<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
--></style>
</head>
<body class='hmmessage'>
BITS 16, I know, but I like BITS 8.<BR>
Using integer directly is viable and efficient, but I doubt worthwhile in this context.<BR>
If they must be abstracted, I think a comment next to them giving their actual values would be good.<BR>
I was tempted to change all the bitsize(x) and bitsize(x) - 1 expressions to 8 and 7.<BR>
So much little layering here and there, can really slow down one's reading comprehension..<BR>
<BR>
- Jay<BR><BR> <BR>
<HR id=stopSpelling>
From: hosking@cs.purdue.edu<BR>Date: Wed, 10 Feb 2010 12:44:23 -0500<BR>To: jay.krell@cornell.edu<BR>CC: m3commit@elegosoft.com<BR>Subject: Re: [M3commit] CVS Update: cm3<BR><BR><BASE>Changes coming that should make life easier...
<DIV><BR></DIV>
<DIV>(HIstorically, the representation was BITS 16 FOR [0..16_FFFF] instead of the current BITS 8 FOR [0..16_FF])<BR><BR>
<DIV>
<DIV>On 10 Feb 2010, at 11:22, Jay K wrote:</DIV><BR class=ecxApple-interchange-newline>
<BLOCKQUOTE><SPAN style="TEXT-TRANSFORM: none; TEXT-INDENT: 0px; BORDER-COLLAPSE: separate; FONT: medium Helvetica; WHITE-SPACE: normal; LETTER-SPACING: normal; WORD-SPACING: 0px" class=ecxApple-style-span>
<DIV style="FONT-FAMILY: Verdana; FONT-SIZE: 10pt" class=ecxhmmessage>You really anticipate it every changing?<BR>Ever having machines that can't address bytes?<BR>I know they used to exist, but it has been several decades.<BR>The world's vast swaths of C, C++, Java, C# won't port easily to them.<BR>If I understood you, you argued that it is clearly IByte, so I should<BR>feel free to make up my own constants outside of TInt.i3/TWord.i3.<BR>Though so far I haven't.<BR> <BR> - Jay<BR><BR> <BR>> From:<SPAN class=ecxApple-converted-space> </SPAN><A href="mailto:hosking@cs.purdue.edu">hosking@cs.purdue.edu</A><BR>> Date: Wed, 10 Feb 2010 10:25:31 -0500<BR>> To:<SPAN class=ecxApple-converted-space> </SPAN><A href="mailto:jkrell@elego.de">jkrell@elego.de</A><BR>> CC:<SPAN class=ecxApple-converted-space> </SPAN><A href="mailto:m3commit@elegosoft.com">m3commit@elegosoft.com</A><BR>> Subject: Re: [M3commit] CVS Update: cm3<BR>><SPAN class=ecxApple-converted-space> </SPAN><BR>> On 10 Feb 2010, at 16:13, Jay Krell wrote:<BR>><SPAN class=ecxApple-converted-space> </SPAN><BR>> > CVSROOT: /usr/cvs<BR>> > Changes by: jkrell@birch. 10/02/10 16:13:30<BR>> ><SPAN class=ecxApple-converted-space> </SPAN><BR>> > Modified files:<BR>> > cm3/m3-sys/m3middle/src/: TWord.m3<SPAN class=ecxApple-converted-space> </SPAN><BR>> ><SPAN class=ecxApple-converted-space> </SPAN><BR>> > Log message:<BR>> > just cleanup a little: make Mask and Base clear and not obfuscated, don't do the same compare twice in a row in xEQ (which should be plain EQ when I'm confident in its correctness, currently what EQ does is compute like three values and verify they all match)<BR>><SPAN class=ecxApple-converted-space> </SPAN><BR>> Mask and Base are computed like that so it is easy to change the underlying representation of Target.Int. We might not have IByte, but instead have IWord!<BR>><SPAN class=ecxApple-converted-space> </SPAN><BR></DIV></SPAN></BLOCKQUOTE></DIV><BR></DIV> </body>
</html>