<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 29, 2017 at 11:01 AM, Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Nicolas, Hi All,<div><br></div><div>   this fixes start up of the Cogit on Win64.  We should add Win64 Cog to the build matrix.  Full testing still required, but at least the system starts up and can be interacted with.</div></div></blockquote><div><br></div><div>The two issues running the tests highlight are</div><div><br></div><div>a) all the SqueakSSLTests fail (primitiveSSLCreate fails)</div><div>b) testWinVM3ButtonMousePreference fails (but I think this is a trivial issue to do with building an ini file; I ran the SqueakConsole.exe VM).</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 29, 2017 at 10:44 AM, GitHub <span dir="ltr"><<a href="mailto:noreply@github.com" target="_blank">noreply@github.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br>  Branch: refs/heads/Cog<br>
  Home:   <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm" rel="noreferrer" target="_blank">https://github.com/OpenSmallt<wbr>alk/opensmalltalk-vm</a><br>
  Commit: 0364ee70e95e68d2acea04067147f1<wbr>02b1510ce0<br>
      <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/0364ee70e95e68d2acea04067147f102b1510ce0" rel="noreferrer" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm/commit/036<wbr>4ee70e95e68d2acea04067147f102b<wbr>1510ce0</a><br>
  Author: Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>><br>
  Date:   2017-05-29 (Mon, 29 May 2017)<br>
<br>
  Changed paths:<br>
    M nsspur64src/vm/cogit.h<br>
    M nsspur64src/vm/cogitX64SysV.c<br>
    M nsspur64src/vm/cogitX64WIN64.c<br>
    M nsspursrc/vm/cogit.h<br>
    M nsspursrc/vm/cogitARMv5.c<br>
    M nsspursrc/vm/cogitIA32.c<br>
    M nsspursrc/vm/cogitMIPSEL.c<br>
    M spur64src/vm/cogit.h<br>
    M spur64src/vm/cogitX64SysV.c<br>
    M spur64src/vm/cogitX64WIN64.c<br>
    M spurlowcode64src/vm/cogit.h<br>
    M spurlowcode64src/vm/cogitX64Sy<wbr>sV.c<br>
    M spurlowcode64src/vm/cogitX64WI<wbr>N64.c<br>
    M spurlowcodesrc/vm/cogit.h<br>
    M spurlowcodesrc/vm/cogitARMv5.c<br>
    M spurlowcodesrc/vm/cogitIA32.c<br>
    M spurlowcodesrc/vm/cogitMIPSEL.<wbr>c<br>
    M spursista64src/vm/cogit.h<br>
    M spursista64src/vm/cogitX64SysV<wbr>.c<br>
    M spursista64src/vm/cogitX64WIN6<wbr>4.c<br>
    M spursistasrc/vm/cogit.h<br>
    M spursistasrc/vm/cogitARMv5.c<br>
    M spursistasrc/vm/cogitIA32.c<br>
    M spursistasrc/vm/cogitMIPSEL.c<br>
    M spursrc/vm/cogit.h<br>
    M spursrc/vm/cogitARMv5.c<br>
    M spursrc/vm/cogitIA32.c<br>
    M spursrc/vm/cogitMIPSEL.c<br>
    M src/plugins/FilePlugin/FilePlu<wbr>gin.c<br>
    M src/vm/cogit.h<br>
    M src/vm/cogitARMv5.c<br>
    M src/vm/cogitIA32.c<br>
    M src/vm/cogitMIPSEL.c<br>
<br>
  Log Message:<br>
  -----------<br>
  CogVM source as per VMMaker.oscog-eem.2227<br>
<br>
Cogit:<br>
Use genMarshallNArgs:arg:arg:arg:a<wbr>rg: when compiling an interpreter primitive<br>
call to allow adding the shadow parameter space on Win64.<br>
<br>
Remove a couple of unused and dubious methods in the CogAbstractInstruction<br>
hierarchy.  Inline arg removal to match inlining of arg marshalling, since<br>
removal is either no or one instruction.<br>
<br>
Fix generation of JumpR on x64.  I forgot a rex byte.  This doesn't affect SysV<br>
because we only ever jump through ClassReg (in an open PIC dispatch) which is<br>
RCX (1) on SysV but R8 (8) on Win64.<br>
With the fix to JumpR a Win64 image simulates correctly (with smashing of the<br>
shadow parameter space).<br>
<br>
FilePlugin:<br>
Incorporate Monty's file plugin changes: add a new primitiveFileOpenNew to open an unexisting file atomically without race condition.<br>
<br>
<br>
<br></blockquote></div><span class="gmail-HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div class="gmail-m_-834288595752231801gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_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>