summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBob Weinand <bobwei9@hotmail.com>2015-10-05 21:52:09 +0200
committerBob Weinand <bobwei9@hotmail.com>2015-10-05 21:52:09 +0200
commit0fbd3e5fe7c2df5f8eb8cf2f217902f80a9fd894 (patch)
treea61123b559f9a90d43d244e394b3fad6d3906b85
parent8c33bdb976e957ea67ebffd424e0a133a79c6ebe (diff)
downloadphp-git-0fbd3e5fe7c2df5f8eb8cf2f217902f80a9fd894.tar.gz
Speed up self::$property access by 20%
-rw-r--r--Zend/zend_compile.c15
1 files changed, 12 insertions, 3 deletions
diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
index 6d8f794ae4..56d946baaf 100644
--- a/Zend/zend_compile.c
+++ b/Zend/zend_compile.c
@@ -2368,12 +2368,21 @@ zend_op *zend_compile_static_prop_common(znode *result, zend_ast *ast, uint32_t
znode class_node, prop_node;
zend_op *opline;
+ uint32_t fetch_type = class_ast->kind == ZEND_AST_ZVAL ? zend_get_class_fetch_type_ast(class_ast) : -1;
- if (zend_is_const_default_class_ref(class_ast)) {
+ if (fetch_type == ZEND_FETCH_CLASS_DEFAULT) {
class_node.op_type = IS_CONST;
ZVAL_STR(&class_node.u.constant, zend_resolve_class_name_ast(class_ast));
} else {
- zend_compile_class_ref(&class_node, class_ast, 1);
+ /* we do not have any reference to the extends_ast/name of parent at compile-time, hence only resolving self:: for now */
+ if (zend_is_scope_known() && fetch_type == ZEND_FETCH_CLASS_SELF) {
+ zend_ensure_valid_class_fetch_type(fetch_type);
+
+ class_node.op_type = IS_CONST;
+ ZVAL_STR_COPY(&class_node.u.constant, CG(active_class_entry)->name);
+ } else {
+ zend_compile_class_ref(&class_node, class_ast, 1);
+ }
}
zend_compile_expr(&prop_node, prop_ast);
@@ -3315,7 +3324,7 @@ void zend_compile_static_call(znode *result, zend_ast *ast, uint32_t type) /* {{
class_node.op_type = IS_CONST;
ZVAL_STR_COPY(&class_node.u.constant, CG(active_class_entry)->name);
} else {
- opline = zend_compile_class_ref(&class_node, class_ast, 1);
+ zend_compile_class_ref(&class_node, class_ast, 1);
}
}