Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion gc/default/default.c
Original file line number Diff line number Diff line change
Expand Up @@ -4377,7 +4377,7 @@ gc_grey(rb_objspace_t *objspace, VALUE obj)
static inline void
gc_mark_check_t_none(rb_objspace_t *objspace, VALUE obj)
{
if (RB_UNLIKELY(RB_TYPE_P(obj, T_NONE))) {
if (RB_UNLIKELY(BUILTIN_TYPE(obj) == T_NONE)) {
enum {info_size = 256};
char obj_info_buf[info_size];
rb_raw_obj_info(obj_info_buf, info_size, obj);
Expand Down
4 changes: 2 additions & 2 deletions test/ruby/test_namespace.rb
Original file line number Diff line number Diff line change
Expand Up @@ -552,7 +552,7 @@ def test_prelude_gems_and_loaded_features

# No additional warnings except for experimental warnings
assert_includes error.join("\n"), EXPERIMENTAL_WARNINGS
assert_equal error.size, 2
assert_equal 2, error.size

assert_includes output.grep(/^before:/).join("\n"), '/bundled_gems.rb'
assert_includes output.grep(/^before:/).join("\n"), '/error_highlight.rb'
Expand All @@ -574,7 +574,7 @@ def test_prelude_gems_and_loaded_features_with_disable_gems
end;

assert_includes error.join("\n"), EXPERIMENTAL_WARNINGS
assert_equal error.size, 2
assert_equal 2, error.size

refute_includes output.grep(/^before:/).join("\n"), '/bundled_gems.rb'
refute_includes output.grep(/^before:/).join("\n"), '/error_highlight.rb'
Expand Down
2 changes: 1 addition & 1 deletion zjit/bindgen/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ fn main() {
.allowlist_function("rb_singleton_class")
.allowlist_function("rb_define_class")
.allowlist_function("rb_class_get_superclass")
.allowlist_function("rb_gc_disable_no_rest")
.allowlist_function("rb_gc_disable")
.allowlist_function("rb_gc_enable")
.allowlist_function("rb_gc_mark")
.allowlist_function("rb_gc_mark_movable")
Expand Down
4 changes: 2 additions & 2 deletions zjit/src/backend/arm64/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -692,10 +692,10 @@ impl Assembler {
/// need to be split with registers after `alloc_regs`, e.g. for `compile_side_exits`, so this
/// splits them and uses scratch registers for it.
fn arm64_split_with_scratch_reg(mut self) -> Assembler {
let mut iterator = self.insns.into_iter().enumerate().peekable();
let iterator = self.insns.into_iter().enumerate().peekable();
let mut asm = Assembler::new_with_label_names(take(&mut self.label_names), self.live_ranges.len(), true);

while let Some((_, mut insn)) = iterator.next() {
for (_, mut insn) in iterator {
match &mut insn {
// For compile_side_exits, support splitting simple C arguments here
Insn::CCall { opnds, .. } if !opnds.is_empty() => {
Expand Down
2 changes: 1 addition & 1 deletion zjit/src/cruby.rs
Original file line number Diff line number Diff line change
Expand Up @@ -897,7 +897,7 @@ where
// 2. If we yield to the GC while compiling, it re-enters our mark and update functions.
// This breaks `&mut` exclusivity since mark functions derive fresh `&mut` from statics
// while there is a stack frame below it that has an overlapping `&mut`. That's UB.
let gc_disabled_pre_call = unsafe { rb_gc_disable_no_rest() }.test();
let gc_disabled_pre_call = unsafe { rb_gc_disable() }.test();

let ret = match catch_unwind(func) {
Ok(result) => result,
Expand Down
2 changes: 1 addition & 1 deletion zjit/src/cruby_bindings.inc.rs

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.