mmtk / mmtk-core

Memory Management ToolKit
https://www.mmtk.io
Other
444 stars 76 forks source link

Pre GC sanity checking #1311

Open qinsoon opened 1 month ago

qinsoon commented 1 month ago

Our current sanity checking happens after the main GC. This becomes less useful for binding developers if they fail in the GC and the sanity checking has no chance to run.

It will be helpful if we can do sanity checking before a GC. The best timing to do so would be after Prepare and before Closure. We need to confirm if we have the complete root set at this point.

k-sareen commented 1 month ago

I thought Angus had already implemented something very similar? We should try and upstream it if possible.