Conversation
|
@mhashizume Could you try this patch with your environment? Unfortunately, I don't have 32bit platform. |
There was a problem hiding this comment.
Pull request overview
This PR addresses RubyGems SafeMarshal memory pressure during large array conversions by ensuring AST array elements are eligible for GC as they’re consumed, targeting the memory exhaustion reported in #9368.
Changes:
- Clear each visited
Elements::Arrayentry (elements[idx] = nil) after converting it to a Ruby object to release AST nodes earlier.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
I'm still seeing the same behavior with this patch applied |
|
@mhashizume Thanks. I don't have target platform so it's just guesswork, but I'll think about it. |
Fixes #9368