{"action":"created","issue":{"url":"https://api.github.com/repos/slavapestov/factor/issues/1315","labels_url":"https://api.github.com/repos/slavapestov/factor/issues/1315/labels{/name}","comments_url":"https://api.github.com/repos/slavapestov/factor/issues/1315/comments","events_url":"https://api.github.com/repos/slavapestov/factor/issues/1315/events","html_url":"https://github.com/slavapestov/factor/pull/1315","id":74784467,"number":1315,"title":"Gc fixes afl part4","user":{"login":"bjourne","id":142475,"avatar_url":"https://avatars.githubusercontent.com/u/142475?v=3","gravatar_id":"","url":"https://api.github.com/users/bjourne","html_url":"https://github.com/bjourne","followers_url":"https://api.github.com/users/bjourne/followers","following_url":"https://api.github.com/users/bjourne/following{/other_user}","gists_url":"https://api.github.com/users/bjourne/gists{/gist_id}","starred_url":"https://api.github.com/users/bjourne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjourne/subscriptions","organizations_url":"https://api.github.com/users/bjourne/orgs","repos_url":"https://api.github.com/users/bjourne/repos","events_url":"https://api.github.com/users/bjourne/events{/privacy}","received_events_url":"https://api.github.com/users/bjourne/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-09T23:38:22Z","updated_at":"2015-05-10T03:02:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/slavapestov/factor/pulls/1315","html_url":"https://github.com/slavapestov/factor/pull/1315","diff_url":"https://github.com/slavapestov/factor/pull/1315.diff","patch_url":"https://github.com/slavapestov/factor/pull/1315.patch"},"body":"Here is another pr to try and fix #1289. I've created a new vocab `compiler.cfg.stacks.padding` which more precisely traces which stack locations are live so you get more accurate gc maps. It supersedes `compiler.cfg.stacks.map`, but I've kept the old one in the history so it's easy to go back to that one if something doesn't work. But it should work. :) You probably need new boot images too. \r\n\r\nI've also written a fix for the \"Generic word lookup-base-pointer*...\" test failure that the build machines got. And some general improvements to the cpp code too. "},"comment":{"url":"https://api.github.com/repos/slavapestov/factor/issues/comments/100576269","html_url":"https://github.com/slavapestov/factor/pull/1315#issuecomment-100576269","issue_url":"https://api.github.com/repos/slavapestov/factor/issues/1315","id":100576269,"user":{"login":"bjourne","id":142475,"avatar_url":"https://avatars.githubusercontent.com/u/142475?v=3","gravatar_id":"","url":"https://api.github.com/users/bjourne","html_url":"https://github.com/bjourne","followers_url":"https://api.github.com/users/bjourne/followers","following_url":"https://api.github.com/users/bjourne/following{/other_user}","gists_url":"https://api.github.com/users/bjourne/gists{/gist_id}","starred_url":"https://api.github.com/users/bjourne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjourne/subscriptions","organizations_url":"https://api.github.com/users/bjourne/orgs","repos_url":"https://api.github.com/users/bjourne/repos","events_url":"https://api.github.com/users/bjourne/events{/privacy}","received_events_url":"https://api.github.com/users/bjourne/received_events","type":"User","site_admin":false},"created_at":"2015-05-10T03:02:04Z","updated_at":"2015-05-10T03:02:04Z","body":"Fixed some small things. The `emit-call-block` is right because I've added a height slot to `##call`. You need it to know how much a call increases or decreases the datastack. It's not needed in `%call` as you noted, but the parameters given to the %-word needs to be the same as the slots in the ##-tuple due to some metaprogramming. Yes the idea is that `trace-stack-state2` will become `trace-stack-state` if the new scheme works as it should. "}}