<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi all,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 8, 2019 at 2:17 AM David T. Lewis <<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><br>
On Tue, May 07, 2019 at 03:36:32PM -0700, Eliot Miranda wrote:<br><br>> In VMMaker.oscog-GAP.2533 I would prefer to see printString instead of<br>
> asString.  asString is a horrible hack.  printString says what we mean.<br></blockquote><div><br></div><div>Sure. I've submitted another version to the inbox:</div><div><br></div><div><div>Name: VMMaker.oscog-GAP.2534</div><div>Author: GAP</div><div>Time: 9 May 2019, 5:12:22.055089 pm</div><div>UUID: cd6aae16-6f20-4ddb-9328-64b19b413cb1</div><div>Ancestors: VMMaker.oscog-eem.2532</div><div><br></div><div>Convert numeric constants to strings for concatenation using #printString.</div><div>Required for Pharo compatibility, where the arguments of concatenation are not automatically coerced to strings.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ==== Patch 1 ====<br>
> > Name: VMMaker.oscog-GAP.2533<br>
> > Author: GAP<br>
> > Time: 7 May 2019, 10:57:16.034354 am<br>
> > UUID: 3209319f-60f3-4586-8d03-cfac3c9fddee<br>
> > Ancestors: VMMaker.oscog-eem.2532<br>
> ><br>
> > Convert numeric constants to strings for concatenation.<br>
> > Required for Pharo compatibility, where the arguments of concatenation are<br>
> > not automatically coerced to strings.<br>
> ><br>
> <br>
> I trust you on these.  I'm not really in a position to test this out right<br>
> now.  I suggest you push the  VMMakerCompatibilityForPharo6 packages to<br>
> VMMaker as soon as you'd like.  You can write VMMaker right?<br>
> <br>
<br>
+1<br>
<br>
Guille, if you want commit access to the repo, you can make an account<br>
on <a href="http://source.squeak.org" rel="noreferrer" target="_blank">source.squeak.org</a> and I'll add you the the VMMaker project. Or if<br>
it's easier, just say which packages you are ready to move and either<br>
I or Eliot (or any of the others with commit access) will move it to<br>
VMMaker for you.<br>
<br>
I note that VMMaker.oscog-GAP.2533 package is part of the main<br>
VMMaker.oscog package. If you need to have that tested on Squeak I<br>
can probably do that for you within the next couple of days if no<br>
one else gets to it first.<br></blockquote><div><br></div><div>Well, the change I did in that package was fairly small, and I'm using squeak to generate also some parts of the VM to compare to what I generate from Pharo.</div><div>So, I'd say I've tried it myself already.</div><div><br></div><div>For the Pharo specific package, I've already uploaded it to the VMMaker repository.</div><div>I had already rights to do it :).</div><div><br></div><div><div>Name: VMMakerCompatibilityForPharo6-GuillermoPolito.11</div><div>Author: GuillermoPolito</div><div>Time: 9 May 2019, 5:15:50.00473 pm</div><div>UUID: e1a685a9-d744-0d00-b3c1-ad600f1a7e9a</div><div>Ancestors: VMMakerCompatibilityForPharo6-GuillermoPolito.10</div><div><br></div></div><div>With these changes I can already compile and run Cog and almost all tests pass :)</div><div><br></div><div><div><img src="cid:ii_jvgsq2gp0" alt="image.png" width="425" height="401"><br></div></div><div><br></div><div>I have also a couple dozen tests about AST translation and C generation, mainly about those parts that are covered differently between Pharo's and Squeak's ASTs.</div><div>Would you like that I push them inside the VMMaker package or make a separate package?</div><div><br></div><div>Thanks!</div><div>Guille</div></div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div></div></div></div></div></div></div>