<div dir="ltr"><div><div><div><div><div><div><div><div><br></div>Hi again,<br></div>I&#39;ve noticed the COG code currently branches on objectMemory wordSize = 8 condition,<br></div>to either load self positive64BitValueOf: integerArgument.<br></div>or to load self positive32BitValueOf: integerArgument.<br><br></div>Of course, this could be resolved to a single<br>    integerArgument := self positiveMachineIntegerValueOf: integerArgument<br>which will care of the branch by itself...<br></div><div>That&#39;s what I did.<br></div><div><br></div>But maybe the intention was rather to deal with signed64BitValueOf in the 64 bit branch like what was done to primitiveBitShift?<br><br>Of course, all this is assuming that the machine uses a 2-complement implementation, otherwise it won&#39;t match the Smalltalk semantic.<br>That&#39;s not so rare ;) - but theoretically not fully portable - contrarily to actual strategy which is fully portable.<br></div><div>Since other parts of the VM also rely on such specific, maybe we can trade a little portability tribute for the sake of speed.<br></div><div><br>Thoughts?<br></div><br></div>Nicolas<br></div>