Skip to content

Commit

Permalink
RJIT: Just skip generating code for aarch64/arm64 (#9221)
Browse files Browse the repository at this point in the history
  • Loading branch information
k0kubun committed Dec 13, 2023
1 parent c83a648 commit 0f1c7e3
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
10 changes: 5 additions & 5 deletions configure.ac
Expand Up @@ -3916,26 +3916,26 @@ AC_SUBST(CARGO_BUILD_ARGS)dnl for selecting Rust build profiles
AC_SUBST(YJIT_LIBS)dnl for optionally building the Rust parts of YJIT
AC_SUBST(YJIT_OBJ)dnl for optionally building the C parts of YJIT
dnl Currently, RJIT only supports Unix x86_64 platforms.
dnl RJIT supports only x86_64 platforms, but allows arm64/aarch64 for custom JITs.
RJIT_TARGET_OK=no
AS_IF([test "$cross_compiling" = no],
AS_CASE(["$target_cpu-$target_os"],
[*android*], [
RJIT_TARGET_OK=no
],
[x86_64-darwin*], [
[arm64-darwin*|aarch64-darwin*|x86_64-darwin*], [
RJIT_TARGET_OK=yes
],
[x86_64-*linux*], [
[arm64-*linux*|aarch64-*linux*|x86_64-*linux*], [
RJIT_TARGET_OK=yes
],
[x86_64-*bsd*], [
[arm64-*bsd*|aarch64-*bsd*|x86_64-*bsd*], [
RJIT_TARGET_OK=yes
]
)
)
dnl Build RJIT on Unix x86_64 platforms or if --enable-rjit is specified.
dnl Build RJIT on supported platforms or if --enable-rjit is specified.
AC_ARG_ENABLE(rjit,
AS_HELP_STRING([--enable-rjit],
[enable pure-Ruby JIT compiler. enabled by default on Unix x86_64 platforms]),
Expand Down
8 changes: 8 additions & 0 deletions lib/ruby_vm/rjit/compiler.rb
Expand Up @@ -59,6 +59,7 @@ def initialize
# @param iseq `RubyVM::RJIT::CPointer::Struct_rb_iseq_t`
# @param cfp `RubyVM::RJIT::CPointer::Struct_rb_control_frame_t`
def compile(iseq, cfp)
return unless supported_platform?
pc = cfp.pc.to_i
jit = JITState.new(iseq:, cfp:)
asm = Assembler.new
Expand Down Expand Up @@ -505,5 +506,12 @@ def assert(cond)
raise "'#{cond.inspect}' was not true"
end
end

def supported_platform?
return @supported_platform if defined?(@supported_platform)
@supported_platform = RUBY_PLATFORM.match?(/x86_64/).tap do |supported|
warn "warning: RJIT does not support #{RUBY_PLATFORM} yet" unless supported
end
end
end
end

0 comments on commit 0f1c7e3

Please sign in to comment.