Feature #13494: [PATCH] pack rb_method_definition_t struct

• Status: Closed
• Assignee: normalperson (Eric Wong)
• Target version: -

[normalperson (Eric Wong)]

We only have 12 method types, so 4 bits is enough for
rb_method_type_t.

Size reductions:

- x86-64     48 => 40 bytes
- x86        28 => 24 bytes

* method.h (enum method_optimized_type): split out for CPP
  (struct rb_method_definition struct): pack on unaligned systems
  (rb_method_definition_t): split typedef to help ctags

[shyouhei (Shyouhei Urabe)]

• Status changed from Open to Assigned

[ko1 (Koichi Sasada)]

Sorry for late response.

No problem.

Thanks,
Koichi

[ko1 (Koichi Sasada)]

• Assignee changed from ko1 (Koichi Sasada) to normalperson (Eric Wong)

• Status changed from Assigned to Closed

Applied in changeset trunk|r58784.

---

method.h: pack rb_method_definition_t struct

We only have 12 method types, so 4 bits is enough for
rb_method_type_t.

Size reductions:

• x86-64 48 => 40 bytes
• x86 28 => 24 bytes

• method.h (enum method_optimized_type): split out for CPP
  (struct rb_method_definition struct): pack on unaligned systems
  (rb_method_definition_t): split typedef to help ctags
  [ruby-core:81236] [Feature #13494]