Skip to content

[pull] master from ruby:master#904

Merged
pull[bot] merged 4 commits intoturkdevops:masterfrom
ruby:master
Apr 2, 2026
Merged

[pull] master from ruby:master#904
pull[bot] merged 4 commits intoturkdevops:masterfrom
ruby:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 2, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

tekknolagi and others added 4 commits April 1, 2026 16:56
…nstruction (#16634)

We have global regalloc now and don't need this manual threading.
pm_parse_process initializes the index_lookup_table but nothing seems to
use it after it has been allocated. However, pm_compile_scope_node will
overwrite the index_lookup_table and cause it to leak memory. This can
be seen during bootup with the following memory leaks reported by ASAN:

    #0 0x60dba31b7af3 in malloc
    #1 0x60dba32e0718 in rb_gc_impl_malloc gc/default/default.c:8287:5
    #2 0x60dba32c7aa7 in ruby_xmalloc_body gc.c:5373:12
    #3 0x60dba32c4a54 in ruby_xmalloc gc.c:5355:34
    #4 0x60dba3260314 in pm_index_lookup_table_init_heap prism_compile.h:89:29
    #5 0x60dba3209388 in pm_parse_process prism_compile.c:11366:5
…k HIR construction (#16634)"

This reverts commit 77b803c.
There is a new NULL dereference in tool/test/testunit/test_sorting.rb
that seem to have started with this commit.
@pull pull bot locked and limited conversation to collaborators Apr 2, 2026
@pull pull bot added the ⤵️ pull label Apr 2, 2026
@pull pull bot merged commit f0c7367 into turkdevops:master Apr 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants