mmtk / ruby

Fork of The Ruby Programming Language [mirror], with added support for MMTk
https://www.ruby-lang.org/
Other
0 stars 1 forks source link

Remove unnecessary include #66

Closed eileencodes closed 5 months ago

eileencodes commented 5 months ago

Previously there was code that conditionally changed with MMTK but that code was removed in the last sync with master.

That code was changed upstream in https://github.com/ruby/ruby/pull/10358. Since there are no if USE_MMTK calls in this file, we no longer need to include internal/mmtk_support.h.

wks commented 5 months ago

Thank you for pointing it out. I have merged this change, but I think I will soon add it back when I try to remove the underlying xmalloc buffer of non-embedded Object. I tried it once, but it was not very profitable, and I had some issues with object shape.