Ruby » Ruby master

Remove unnecessary assignment to ast->body.line_count

This patch removes a code that assigns -1 to ast->body.line_count because, at least as of now, it looks not necessary.
I made this commit atomically revertable if I was wrong.

Relevant commits...¶

Fixed a value specified for OPT_THREADED_CODE (#10657)

Values defined for OPT_THREADED_CODE are 0,1,2. However, 1,2,3 are set in workflow. It seems that case 3 does not exist, so 0 is specified instead.

Co-authored-by: jinroq jinroq@users.noreply.github.com

Revert "YJIT: Try splitting getlocal/setlocal blocks (#10648)"

This reverts commit ab228bd0844758a1c444e39030c153874adf9120.

Add line_count field to rb_ast_body_t

This patch adds int line_count field to rb_ast_body_t structure.
Instead, we no longer cast script_lines to Fixnum.

Background¶

Ref https://github.com/ruby/ruby/pull/10618

In the PR above, we have decoupled IMEMO from rb_ast_t....

Lrama v0.6.6

YJIT: Correct signature of rb_yjit_root_mark()

Even though unused, it's supposed to take a pointer like the C side
expects.

YJIT: Stop asserting rb_objspace_markable_object_p()

Because of the way things are sequenced, it doesn't work properly during
auto-compaction.

YJIT: Fix reference update for Invariants::no_ep_escape_iseqs

Previously, the update was done in the ISEQ callback. That effectively
never updated anything because the callback itself is given an intact
reference, so it could update its content, and rb_gc_location(iseq)...

Revert "Use -v to investigate which test is stuck"

This reverts commit 444553b528a54a97e539de212ffc4c6466a6db20.

At least it should no longer timeout.

Avoid overriding GNUMAKEFLAGS's -j

.github/actions/setup/directories/action.yml sets GNUMAKEFLAGS. Having
-j here is rather harmful.

Partly reverts f8dad616c2ee2d83b3162da8d86865b0f2a782de.