From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-2.6 required=3.0 tests=AWL,BAYES_00, DKIM_ADSP_CUSTOM_MED,FORGED_GMAIL_RCVD,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY shortcircuit=no autolearn=no autolearn_force=no version=3.4.2 Received: from neon.ruby-lang.org (neon.ruby-lang.org [221.186.184.75]) by dcvr.yhbt.net (Postfix) with ESMTP id 5801B1F4B4 for ; Thu, 31 Dec 2020 07:47:26 +0000 (UTC) Received: from neon.ruby-lang.org (localhost [IPv6:::1]) by neon.ruby-lang.org (Postfix) with ESMTP id 75E5C120B49; Thu, 31 Dec 2020 16:46:35 +0900 (JST) Received: from xtrwkhkc.outbound-mail.sendgrid.net (xtrwkhkc.outbound-mail.sendgrid.net [167.89.16.28]) by neon.ruby-lang.org (Postfix) with ESMTPS id C564A120ADD for ; Thu, 31 Dec 2020 16:46:33 +0900 (JST) Received: by filterdrecv-p3mdw1-7474cd8bfd-d6gbs with SMTP id filterdrecv-p3mdw1-7474cd8bfd-d6gbs-19-5FED8202-18 2020-12-31 07:47:14.700221304 +0000 UTC m=+1759496.965863878 Received: from herokuapp.com (unknown) by geopod-ismtpd-6-9 (SG) with ESMTP id 4O9d01TARPqr3BZLlxfe9g for ; Thu, 31 Dec 2020 07:47:14.616 +0000 (UTC) Date: Thu, 31 Dec 2020 07:47:14 +0000 (UTC) From: takashikkbn@gmail.com Message-ID: References: Mime-Version: 1.0 X-Redmine-MailingListIntegration-Message-Ids: 77736 X-Redmine-Project: ruby-master X-Redmine-Issue-Tracker: Feature X-Redmine-Issue-Id: 17490 X-Redmine-Issue-Author: k0kubun X-Redmine-Sender: k0kubun X-Mailer: Redmine X-Redmine-Host: bugs.ruby-lang.org X-Redmine-Site: Ruby Issue Tracking System X-Auto-Response-Suppress: All Auto-Submitted: auto-generated X-SG-EID: =?us-ascii?Q?9bN4wU0cxWeLAEOz6NELGbwCNAokUg6cnjH8nvx=2FqgY5tOi4wkQXlAsiSU66Ik?= =?us-ascii?Q?5eZW=2FrpHrkxBF8gOCzgDjkkgXpkVhSPsOB7un8B?= =?us-ascii?Q?ySpx24poimx11J6vGZibdtWx0zBjjw+Xsj981sn?= =?us-ascii?Q?p=2F=2FrtaEQG1GdTt=2FRN3Sm8jR7dSQHIDSTLMjhoNk?= =?us-ascii?Q?GBOtUc4wPK16xM6Rb5d8piUIz6wPY8zISvpPS2b?= =?us-ascii?Q?tMQNVQr5oQtzzWvcE=3D?= To: ruby-core@ruby-lang.org X-Entity-ID: b/2+PoftWZ6GuOu3b0IycA== X-ML-Name: ruby-core X-Mail-Count: 101832 Subject: [ruby-core:101832] [Ruby master Feature#17490] Rename RubyVM::MJIT to RubyVM::JIT X-BeenThere: ruby-core@ruby-lang.org X-Mailman-Version: 2.1.15 Precedence: list Reply-To: Ruby developers List-Id: Ruby developers List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ruby-core-bounces@ruby-lang.org Sender: "ruby-core" Issue #17490 has been updated by k0kubun (Takashi Kokubun). > If we'd consider RubyVM is CRuby-specific, then it seems fine. > However, that's rather unclear, and then the question is what should other Ruby implementations should do RubyVM::MJIT and its methods? > > RubyVM::MJIT.enabled? could become a portable API, but it would need to move somewhere else. > I'm also unsure if it's needed besides MRI tests Didn't you clarify it by yourself at [Feature #15743]? The person who wrote the line that nobu quoted was you. You made it pretty clear that `RubyVM::MJIT` doesn't need to exist in other implementations. > I think Ruby users should anyway not need to use RubyVM::MJIT methods, so I prefer the more precise MJIT name. So, would you suggest always explaining what is "M"JIT in every release note and renaming `--jit` to `--mjit` to approach the naming inconsistency issue explained in this ticket? I'm fine with pasting a link for the former, but I'm reluctant to force users to remember the flag name. > IMHO, a benchmark harness should not need to use a JIT-specific API, so I think pause/resume are only useful for debugging. I agree. None of such benchmarks are supposed to be used for purposes other than testing MRI vs MRI+MJIT. I mean, can't I even put a script to test it? ---------------------------------------- Feature #17490: Rename RubyVM::MJIT to RubyVM::JIT https://bugs.ruby-lang.org/issues/17490#change-89662 * Author: k0kubun (Takashi Kokubun) * Status: Open * Priority: Normal ---------------------------------------- ## Background In my understanding, MJIT is a codename like YARV which many people outside Ruby community are not familiar with, so I've used JIT in NEWS or release notes to avoid explaining the "M" part whenever we release a new version. However, because we have the name "MJIT" in one of our constants, we've had some naming inconsistency. For instance, --jit is not --mjit and it's not consistent. ## Proposal Have the same constant as `RubyVM::JIT`, deprecate `RubyVM::MJIT` from Ruby 3.1, and remove the old one in Ruby 3.2. -- https://bugs.ruby-lang.org/