Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GC clean up #16654

Merged
merged 6 commits into from
Jun 4, 2016
Merged

GC clean up #16654

merged 6 commits into from
Jun 4, 2016

Conversation

yuyichao
Copy link
Contributor

This is on top of #16632 to avoid massive merge conflict...

Main changes are,

  • Delete left over logic from the incremental GC.

    We are making a lot of assumptions about the marking bit and the deleted part should be the easiest part to fix if we want the incremental behavior back at some point...

  • Replace sweep_mask with sweep_full.

    The mask is effectively a bool for the two types of sweep we have.

  • Move more debugging code into gc-debug.c

    This also remove some unnecessary counter increments for the build without these debug options enabled. (the release build might be able to optimize those out automatically).

@yuyichao yuyichao added the GC Garbage collector label May 29, 2016
@yuyichao yuyichao force-pushed the yyc/gc/cleanup branch 2 times, most recently from 4a05f4e to 3fb8728 Compare May 30, 2016 20:26
@yuyichao yuyichao closed this May 30, 2016
@yuyichao yuyichao reopened this May 30, 2016
@yuyichao
Copy link
Contributor Author

Travis really don't want to run this PR somehow?

@vtjnash
Copy link
Member

vtjnash commented May 30, 2016

lgtm so far

@yuyichao yuyichao force-pushed the yyc/gc/cleanup branch 8 times, most recently from 4053084 to b44df31 Compare June 2, 2016 12:54
@yuyichao
Copy link
Contributor Author

yuyichao commented Jun 2, 2016

@nanosoldier runbenchmarks(ALL, vs = ":master")

@nanosoldier
Copy link
Collaborator

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @jrevels

@yuyichao yuyichao force-pushed the yyc/gc/cleanup branch 3 times, most recently from af3e441 to a9c0357 Compare June 3, 2016 12:36
yuyichao added 6 commits June 3, 2016 22:54
* Remove a few unused variables
* Remove unused mark mode
* Remove `SWAP_wr`
…o gc-debug.c

Also move MEMPROFILE to gc-debug.c (still broken)
* Now that the page metadata is accurate, we don't need to backoff when we
  sweep an empty page anymore.
* Remove `MEMDEBUG` in page sweep since it's not used.
@yuyichao yuyichao merged commit 1da68b1 into master Jun 4, 2016
@yuyichao yuyichao deleted the yyc/gc/cleanup branch June 4, 2016 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GC Garbage collector
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants