summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@zend.com>2020-04-08 12:39:24 +0300
committerDmitry Stogov <dmitry@zend.com>2020-04-08 12:39:24 +0300
commit3042b2b2012fa1f701170282e4f7ac55089f79b0 (patch)
treeae35499384696a7892e8457ca74f5b9f0f72c7a0
parent6a67067c21e18a6a613bdc61a70b1a62e0b9a8d2 (diff)
downloadphp-git-3042b2b2012fa1f701170282e4f7ac55089f79b0.tar.gz
Dump information about trace side exits
-rw-r--r--ext/opcache/jit/zend_jit.h1
-rw-r--r--ext/opcache/jit/zend_jit_trace.c56
2 files changed, 56 insertions, 1 deletions
diff --git a/ext/opcache/jit/zend_jit.h b/ext/opcache/jit/zend_jit.h
index aea15ea1b7..e53c88e19c 100644
--- a/ext/opcache/jit/zend_jit.h
+++ b/ext/opcache/jit/zend_jit.h
@@ -84,6 +84,7 @@
#define ZEND_JIT_DEBUG_TRACE_BLACKLIST (1<<17)
#define ZEND_JIT_DEBUG_TRACE_BYTECODE (1<<18)
#define ZEND_JIT_DEBUG_TRACE_TSSA (1<<19)
+#define ZEND_JIT_DEBUG_TRACE_EXIT_INFO (1<<20)
ZEND_EXT_API int zend_jit_op_array(zend_op_array *op_array, zend_script *script);
ZEND_EXT_API int zend_jit_script(zend_script *script);
diff --git a/ext/opcache/jit/zend_jit_trace.c b/ext/opcache/jit/zend_jit_trace.c
index 0f5f1244eb..65b5890021 100644
--- a/ext/opcache/jit/zend_jit_trace.c
+++ b/ext/opcache/jit/zend_jit_trace.c
@@ -457,6 +457,7 @@ static zend_ssa *zend_jit_trace_build_ssa(const zend_op_array *op_array, zend_sc
}
static void zend_jit_dump_trace(zend_jit_trace_rec *trace_buffer, zend_ssa *tssa);
+static void zend_jit_dump_exit_info(zend_jit_trace_info *t);
static zend_always_inline int zend_jit_trace_op_len(const zend_op *opline)
{
@@ -3168,7 +3169,7 @@ static zend_jit_trace_stop zend_jit_compile_root_trace(zend_jit_trace_rec *trace
{
zend_jit_trace_stop ret;
const void *handler;
- zend_jit_trace_info *t;
+ zend_jit_trace_info *t = NULL;
zend_jit_trace_exit_info exit_info[ZEND_JIT_TRACE_MAX_EXITS];
zend_shared_alloc_lock();
@@ -3260,6 +3261,12 @@ exit:
zend_shared_alloc_unlock();
+ if ((ZCG(accel_directives).jit_debug & ZEND_JIT_DEBUG_TRACE_EXIT_INFO) != 0
+ && ret == ZEND_JIT_TRACE_STOP_COMPILED
+ && t->exit_count > 0) {
+ zend_jit_dump_exit_info(t);
+ }
+
return ret;
}
@@ -3499,6 +3506,47 @@ static void zend_jit_dump_trace(zend_jit_trace_rec *trace_buffer, zend_ssa *tssa
}
}
+static void zend_jit_dump_exit_info(zend_jit_trace_info *t)
+{
+ int i, j;
+
+ fprintf(stderr, "---- TRACE %d exit info\n", t->id);
+ for (i = 0; i < t->exit_count; i++) {
+ uint32_t stack_size = t->exit_info[i].stack_size;
+ zend_jit_trace_stack *stack = t->stack_map + t->exit_info[i].stack_offset;
+
+ fprintf(stderr, " exit_%d:", i);
+ if (t->exit_info[i].opline) {
+ // TODO: print exit opline number ???
+ //fprintf(stderr, " %04d/", t->exit_info[i].opline - op_array->opcodes);
+ fprintf(stderr, " XXXX/");
+ } else {
+ fprintf(stderr, " ----/");
+ }
+ if (t->exit_info[i].stack_size) {
+ fprintf(stderr, "%04d/%d", t->exit_info[i].stack_offset, t->exit_info[i].stack_size);
+ } else {
+ fprintf(stderr, "----/0");
+ }
+ for (j = 0; j < stack_size; j++) {
+ zend_uchar type = STACK_TYPE(stack, j);
+ if (type != IS_UNKNOWN) {
+ // TODO: print CV insted of X ???
+ fprintf(stderr, " X%d:", j);
+ if (type == IS_UNDEF) {
+ fprintf(stderr, "undef");
+ } else {
+ fprintf(stderr, "%s", zend_get_type_by_const(type));
+ if (STACK_REG(stack, j) != ZREG_NONE) {
+ fprintf(stderr, "(%s)", zend_reg_name[STACK_REG(stack, j)]);
+ }
+ }
+ }
+ }
+ fprintf(stderr, "\n");
+ }
+}
+
int ZEND_FASTCALL zend_jit_trace_hot_root(zend_execute_data *execute_data, const zend_op *opline)
{
const zend_op *orig_opline;
@@ -3771,6 +3819,12 @@ exit:
zend_shared_alloc_unlock();
+ if ((ZCG(accel_directives).jit_debug & ZEND_JIT_DEBUG_TRACE_EXIT_INFO) != 0
+ && ret == ZEND_JIT_TRACE_STOP_COMPILED
+ && t->exit_count > 0) {
+ zend_jit_dump_exit_info(t);
+ }
+
return ret;
}