Skip to content

Commit e40da2f

Browse files
kocsismatearnaud-lb
andcommitted
Review + test fixes
Co-Authored-By: Arnaud Le Blanc <[email protected]>
1 parent 8624843 commit e40da2f

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

Zend/Optimizer/compact_literals.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -556,6 +556,7 @@ void zend_optimizer_compact_literals(zend_op_array *op_array, zend_optimizer_ctx
556556
break;
557557
case ZEND_ASSIGN_OBJ:
558558
case ZEND_ASSIGN_OBJ_REF:
559+
case ZEND_CLONE_INIT_PROP:
559560
case ZEND_FETCH_OBJ_R:
560561
case ZEND_FETCH_OBJ_W:
561562
case ZEND_FETCH_OBJ_RW:

Zend/zend_language_parser.y

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ static YYSIZE_T zend_yytnamerr(char*, const char*);
279279
%type <ast> attributed_statement attributed_class_statement attributed_parameter
280280
%type <ast> attribute_decl attribute attributes attribute_group namespace_declaration_name
281281
%type <ast> match match_arm_list non_empty_match_arm_list match_arm match_arm_cond_list
282-
%type <ast> clone property_initializer_list non_empty_property_initializer_list property_initializer_expr
282+
%type <ast> clone clone_property_initializer_list non_empty_clone_property_initializer_list clone_property_initializer_expr
283283
%type <ast> enum_declaration_statement enum_backing_type enum_case enum_case_expr
284284
%type <ast> function_name non_empty_member_modifiers
285285

@@ -1583,19 +1583,19 @@ isset_variable:
15831583

15841584
clone:
15851585
T_CLONE expr { $$ = zend_ast_create(ZEND_AST_CLONE, $2, NULL); }
1586-
| T_CLONE expr T_WITH property_initializer_list { $$ = zend_ast_create(ZEND_AST_CLONE, $2, $4); }
1586+
| T_CLONE expr T_WITH clone_property_initializer_list { $$ = zend_ast_create(ZEND_AST_CLONE, $2, $4); }
15871587

1588-
property_initializer_list:
1588+
clone_property_initializer_list:
15891589
'{' '}' { $$ = zend_ast_create_list(0, ZEND_AST_PROPERTY_INITIALIZER_LIST); }
1590-
| '{' non_empty_property_initializer_list possible_comma '}' { $$ = $2; }
1590+
| '{' non_empty_clone_property_initializer_list possible_comma '}' { $$ = $2; }
15911591
;
15921592

1593-
non_empty_property_initializer_list:
1594-
property_initializer_expr { $$ = zend_ast_create_list(1, ZEND_AST_PROPERTY_INITIALIZER_LIST, $1); }
1595-
| non_empty_property_initializer_list ',' property_initializer_expr { $$ = zend_ast_list_add($1, $3); }
1593+
non_empty_clone_property_initializer_list:
1594+
clone_property_initializer_expr { $$ = zend_ast_create_list(1, ZEND_AST_PROPERTY_INITIALIZER_LIST, $1); }
1595+
| non_empty_clone_property_initializer_list ',' clone_property_initializer_expr { $$ = zend_ast_list_add($1, $3); }
15961596
;
15971597

1598-
property_initializer_expr:
1598+
clone_property_initializer_expr:
15991599
identifier ':' expr { $$ = zend_ast_create(ZEND_AST_INITIALIZER_EXPR, $1, $3); }
16001600
| expr T_DOUBLE_ARROW expr { $$ = zend_ast_create(ZEND_AST_INITIALIZER_EXPR, $1, $3); }
16011601
;

ext/opcache/tests/clone_with_exception_handling.phpt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ opcache
66
opcache.enable=1
77
opcache.enable_cli=1
88
opcache.opt_debug_level=0x20000
9-
opcache.optimization_level=0x0CFF
109
--FILE--
1110
<?php
1211

0 commit comments

Comments
 (0)