Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port opt_str_uminus to new backend IR #370

Merged
merged 1 commit into from Aug 5, 2022
Merged

Conversation

fresh-eggs
Copy link

irb(main):009:0' code = %q{
irb(main):010:0'     def foo()
irb(main):011:0'       -"mosaic"
irb(main):012:0'     end
irb(main):013:0'     foo()
irb(main):014:0> }
=> "\n    def foo()\n      -\"mosaic\"\n    end\n    foo()\n"
irb(main):015:0> puts RubyVM::InstructionSequence.compile(code).disassemble
== disasm: #<ISeq:<compiled>@<compiled>:1 (1,0)-(5,9)> (catch: FALSE)
0000 definemethod                           :foo, foo                 (   2)[Li]
0003 putself                                                          (   5)[Li]
0004 opt_send_without_block                 <calldata!mid:foo, argc:0, FCALL|ARGS_SIMPLE>
0006 leave

== disasm: #<ISeq:foo@<compiled>:2 (2,4)-(4,7)> (catch: FALSE)
0000 opt_str_uminus                         "mosaic", <calldata!mid:-@, argc:0, ARGS_SIMPLE>(   3)[LiCa]
0003 leave                                                            (   4)[Re]
make -j miniruby && RUST_BACKTRACE=1 ruby --disable=gems bootstraptest/runner.rb --ruby="./miniruby -I./lib -I. -I.ext/common --disable-gems --yjit-call-threshold=1 --yjit-verify-ctx" bootstraptest/test_yjit_new_backend.rb

[...]

2022-08-05 12:32:22 -0400
Driver is ruby 3.2.0dev (2022-07-22T22:02:24Z yjit_backend_ir 639815ef68) [x86_64-darwin21]
Target is ruby 3.2.0dev (2022-08-05T16:23:34Z opt_str_uminu abb6670c01) +YJIT [x86_64-darwin21]

test_yjit_new_backend.rb  PASS 79

Finished in 2.63 sec

PASS all 79 tests

@maximecb maximecb merged commit 46abb61 into yjit_backend_ir Aug 5, 2022
@maximecb maximecb deleted the opt_str_uminu branch August 5, 2022 17:32
noahgibbs pushed a commit that referenced this pull request Aug 24, 2022
noahgibbs pushed a commit that referenced this pull request Aug 25, 2022
noahgibbs pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 25, 2022
k0kubun pushed a commit that referenced this pull request Aug 26, 2022
noahgibbs pushed a commit that referenced this pull request Aug 26, 2022
k0kubun pushed a commit that referenced this pull request Aug 26, 2022
k0kubun pushed a commit that referenced this pull request Aug 29, 2022
k0kubun pushed a commit that referenced this pull request Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants