mflatt
2017-5-23 13:38:46

You can now use make run ARGS="-l setup" to compile racket and a little more to ".zo" form. The ".zo" file is written to “compiled/<platform>/<file>_rkt.zo” as platform-specific code, for now.


mflatt
2017-5-23 13:40:26

Issues: loading compiled code is currently slow, and the ".zo" files are incompatible with any change to “core”, “thread”, “io”, or “regexp”


samth
2017-5-23 13:40:39

Exciting!


samth
2017-5-23 13:41:17

What is the compiled code the output of?


samth
2017-5-23 13:41:31

The chez compiler, the expander, the schemify pass?


mflatt
2017-5-23 13:41:47

The Chez Scheme compiler


mflatt
2017-5-23 13:42:17

I was aiming for a best-case scenario in load times, but it’s currently 10 times as slow as Racket


mflatt
2017-5-23 13:42:40

I don’t yet know why


samth
2017-5-23 17:17:14

@spall I think you need to rebase before pushing


spall
2017-5-23 17:18:54

@samth why?


samth
2017-5-23 17:19:08

your PR now has 25 commits in it, and a merge conflict


spall
2017-5-23 17:19:33

how will rebasing get rid of those commits?


spall
2017-5-23 17:20:22

Okay there are duplicate commits


samth
2017-5-23 17:20:22

most of those commits are by @mflatt


spall
2017-5-23 17:20:49

yeah probably from when i rebased last week


spall
2017-5-23 17:21:19

how can i fix this?


samth
2017-5-23 17:21:35

rebase again


mflatt
2017-5-23 18:02:16

@spall You have things in a state that I don’t know how to adjust easily; I’ll merge with the latest, then apply a diff of that with respect to the latest, then take that patch to a clean tree and apply it (attributed to your email address)


spall
2017-5-23 18:02:33

@mflatt I’m working on fixing it right now.


mflatt
2017-5-23 18:02:59

Ok!


spall
2017-5-23 18:47:46

@mflatt I think I have fixed it


spall
2017-5-23 18:48:14

Should only see the commits I actually made now.


samth
2017-5-23 18:49:34

btw, @mflatt, you can review the combined commits here: https://github.com/racket/racket7/pull/5/files


samth
2017-5-23 18:49:43

even though it’s multiple commits


spall
2017-5-23 18:49:56

I did try to squash all the (new) related commits


samth
2017-5-23 18:55:35

@spall I added some comments


spall
2017-5-23 18:56:41

@samth I added the extra space to make it super obvious that it was an application


spall
2017-5-23 18:56:47

I can remove them


mflatt
2017-5-23 19:30:18

@spall The current version looks ok; I’ll squash before pushing to help make sure I understand the changes