Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

* vm_insnhelper.h: partly revert r37631 (DTrace support).

  "vm usage information is always collected, so uncomment the
  functions." causes performance impact. [Bug #7370]
  Off course this revert disables related probes.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37796 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information...
commit 4ba69a41d8373bfdc17f54a1d5f16886ad3f6371 1 parent d5d2440
NARUSE, Yui nurse authored

Showing 2 changed files with 10 additions and 1 deletion. Show diff stats Hide diff stats

  1. +7 0 ChangeLog
  2. +3 1 vm_insnhelper.h
7 ChangeLog
... ... @@ -1,3 +1,10 @@
  1 +Thu Nov 22 15:26:02 2012 NARUSE, Yui <naruse@ruby-lang.org>
  2 +
  3 + * vm_insnhelper.h: partly revert r37631 (DTrace support).
  4 + "vm usage information is always collected, so uncomment the
  5 + functions." causes performance impact. [Bug #7370]
  6 + Off course this revert disables related probes.
  7 +
1 8 Thu Nov 22 14:14:36 2012 Akinori MUSHA <knu@iDaemons.org>
2 9
3 10 * re.c (rb_memsearch_ss): Apply performance improvement to short
4 vm_insnhelper.h
@@ -62,12 +62,14 @@ enum {
62 62 extern char ruby_vm_redefined_flag[BOP_LAST_];
63 63 extern VALUE ruby_vm_const_missing_count;
64 64
  65 +#if VM_COLLECT_USAGE_DETAILS
65 66 #define COLLECT_USAGE_INSN(insn) vm_collect_usage_insn(insn)
66 67 #define COLLECT_USAGE_OPERAND(insn, n, op) vm_collect_usage_operand((insn), (n), ((VALUE)(op)))
67 68
68   -#if VM_COLLECT_USAGE_DETAILS
69 69 #define COLLECT_USAGE_REGISTER(reg, s) vm_collect_usage_register((reg), (s))
70 70 #else
  71 +#define COLLECT_USAGE_INSN(insn) /* none */
  72 +#define COLLECT_USAGE_OPERAND(insn, n, op) /* none */
71 73 #define COLLECT_USAGE_REGISTER(reg, s) /* none */
72 74 #endif
73 75

0 comments on commit 4ba69a4

Please sign in to comment.
Something went wrong with that request. Please try again.