From 77293cef91a9aa424c086ae05f03211d9a8a87d3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=8D=9C=E9=83=A8=E6=98=8C=E5=B9=B3?= Date: Fri, 29 May 2020 15:20:57 +0900 Subject: vm_ci_markable: added CIs are created on-the-fly, which increases GC pressure. However they include no references to other objects, and those on-the-fly CIs tend to be short lived. Why not skip allocation of them. In doing so we need to add a flag denotes the CI object does not reside inside of objspace. --- vm_callinfo.h | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) (limited to 'vm_callinfo.h') diff --git a/vm_callinfo.h b/vm_callinfo.h index e938092926..14027fc112 100644 --- a/vm_callinfo.h +++ b/vm_callinfo.h @@ -229,6 +229,23 @@ vm_ci_new_runtime_(ID mid, unsigned int flag, unsigned int argc, const struct rb return vm_ci_new_(mid, flag, argc, kwarg, file, line); } +#define VM_CALLINFO_NOT_UNDER_GC IMEMO_FL_USER0 + +static inline bool +vm_ci_markable(const struct rb_callinfo *ci) +{ + if (! ci) { + return false; /* or true? This is Qfalse... */ + } + else if (vm_ci_packed_p(ci)) { + return true; + } + else { + VM_ASSERT(IMEMO_TYPE_P(ci, imemo_callinfo)); + return ! FL_ANY_RAW((VALUE)ci, VM_CALLINFO_NOT_UNDER_GC); + } +} + typedef VALUE (*vm_call_handler)( struct rb_execution_context_struct *ec, struct rb_control_frame_struct *cfp, -- cgit v1.2.1