mmtk / mmtk-core

Memory Management ToolKit
https://www.mmtk.io
Other
376 stars 69 forks source link

Check number of bits for in header metadata specs #363

Closed qinsoon closed 3 years ago

qinsoon commented 3 years ago

For each MetadataSpec, mmtk-core has assumptions about how many bits it should have. However, the spec is defined by the binding, and they can use any number of bits. Incorrect number of bits may cause MMTk to fail obscurely. We need to fix this. We can do either of the two:

We also should make the assumption and the requirement for each spec very clear in the ObjectModel so binding developers know what they need to provide.

qinsoon commented 3 years ago

We now use a specific type for each kind of metadata in ObjectModel, which predefines the number of bits and the region size. This issue can be closed.