<div dir="ltr">Hi Eliot,<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Second, Ronie, in looking at ssFlushTo:nativeFlushTo: it seems to 
me that it is only ever called with nativeIndex being supplied 
from simNativeStackPtr.  If you agree with me that that is the case 
would you object to me deleting ssFlushTo:<wbr>nativeFlushTo: and implementing ssFlushTo: to include LowcodeVM ifTrue: [self ssNativeFlushTo: <wbr>simNativeStackPtr]?
 This is a little cleaner,  We can always make it more complicated later
 on if required, but right now I'd like to keep things as simple as 
possible.</div></blockquote><div>Sounds fine by me.</div><div><br></div><div>BTW, we should have some talk during one of these days.</div><div><br></div><div>Best regards,</div><div>Ronie<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-01-22 22:47 GMT-03:00 Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Ronie, Hi Clément, Hi All,<div><br></div><div>    one poor design decision I made when writing StackToRegisterMappingCogit was in keeping simSelf, the simulation stack entry for the receiver, separate from simStack, the simulation stack entries for the stack proper.  I'm now at a point where I want to get RegisterAllocatingCogit working properly so that soon Clément and I can merge it into the SistaCogit and get much improved register allocation for Sista/Scorch.  So I'm starting by duplicating <wbr>StackToRegisterMappingCogit and making the changes in the city so I can verify that the existing one and the new one without simSelf or optStatus generate exactly the same code.</div><div><br></div><div>First of all, is that ok with both of you?</div><div><br></div><div>Second, Ronie, in looking at ssFlushTo:nativeFlushTo: it seems to me that it is only ever called with nativeIndex being supplied from simNativeStackPtr.  If you agree with me that that is the case would you object to me deleting ssFlushTo:<wbr>nativeFlushTo: and implementing ssFlushTo: to include LowcodeVM ifTrue: [self ssNativeFlushTo: <wbr>simNativeStackPtr]? This is a little cleaner,  We can always make it more complicated later on if required, but right now I'd like to keep things as simple as possible.</div><div><br><div class="m_-7359328533130183413gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</div></div>
</blockquote></div><br></div>