Skip to content

Commit a4e2987

Browse files
committed
Make ZEND_SHORT_CIRCUITING_CHAIN_ consts no bit flags
1 parent 17953e5 commit a4e2987

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

Zend/zend_compile.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -352,8 +352,9 @@ typedef struct _zend_oparray_context {
352352
/* call through internal function handler. e.g. Closure::invoke() */
353353
#define ZEND_ACC_CALL_VIA_HANDLER ZEND_ACC_CALL_VIA_TRAMPOLINE
354354

355-
#define ZEND_SHORT_CIRCUITING_CHAIN_ISSET (1 << 0)
356-
#define ZEND_SHORT_CIRCUITING_CHAIN_EMPTY (1 << 1)
355+
#define ZEND_SHORT_CIRCUITING_CHAIN_EXPR 0
356+
#define ZEND_SHORT_CIRCUITING_CHAIN_ISSET 1
357+
#define ZEND_SHORT_CIRCUITING_CHAIN_EMPTY 2
357358

358359
char *zend_visibility_string(uint32_t fn_flags);
359360

Zend/zend_vm_def.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7042,7 +7042,7 @@ ZEND_VM_HOT_NOCONST_HANDLER(195, ZEND_JMP_NULL, CONST|TMPVAR|CV, JMP_ADDR)
70427042
} else {
70437043
zval *result = EX_VAR(opline->result.var);
70447044

7045-
if (EXPECTED(opline->extended_value == 0)) {
7045+
if (EXPECTED(opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_EXPR)) {
70467046
ZVAL_NULL(result);
70477047
} else if (opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_ISSET) {
70487048
ZVAL_BOOL(result, 0);

Zend/zend_vm_execute.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4295,7 +4295,7 @@ static ZEND_VM_COLD ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_JMP_NULL_SPEC_CON
42954295
} else {
42964296
zval *result = EX_VAR(opline->result.var);
42974297

4298-
if (EXPECTED(opline->extended_value == 0)) {
4298+
if (EXPECTED(opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_EXPR)) {
42994299
ZVAL_NULL(result);
43004300
} else if (opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_ISSET) {
43014301
ZVAL_BOOL(result, 0);
@@ -13206,7 +13206,7 @@ static ZEND_VM_HOT ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_JMP_NULL_SPEC_TMPV
1320613206
} else {
1320713207
zval *result = EX_VAR(opline->result.var);
1320813208

13209-
if (EXPECTED(opline->extended_value == 0)) {
13209+
if (EXPECTED(opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_EXPR)) {
1321013210
ZVAL_NULL(result);
1321113211
} else if (opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_ISSET) {
1321213212
ZVAL_BOOL(result, 0);
@@ -36591,7 +36591,7 @@ static ZEND_VM_HOT ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_JMP_NULL_SPEC_CV_H
3659136591
} else {
3659236592
zval *result = EX_VAR(opline->result.var);
3659336593

36594-
if (EXPECTED(opline->extended_value == 0)) {
36594+
if (EXPECTED(opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_EXPR)) {
3659536595
ZVAL_NULL(result);
3659636596
} else if (opline->extended_value == ZEND_SHORT_CIRCUITING_CHAIN_ISSET) {
3659736597
ZVAL_BOOL(result, 0);

ext/opcache/Optimizer/zend_inference.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4363,6 +4363,7 @@ int zend_may_throw(const zend_op *opline, const zend_ssa_op *ssa_op, const zend_
43634363
case ZEND_FUNC_GET_ARGS:
43644364
case ZEND_COPY_TMP:
43654365
case ZEND_CHECK_FUNC_ARG:
4366+
case ZEND_JMP_NULL:
43664367
return 0;
43674368
case ZEND_INIT_FCALL:
43684369
/* can't throw, because call is resolved at compile time */
@@ -4426,7 +4427,6 @@ int zend_may_throw(const zend_op *opline, const zend_ssa_op *ssa_op, const zend_
44264427
case ZEND_JMPNZ_EX:
44274428
case ZEND_BOOL:
44284429
case ZEND_JMP_SET:
4429-
case ZEND_JMP_NULL:
44304430
return (t1 & MAY_BE_OBJECT);
44314431
case ZEND_BOOL_XOR:
44324432
return (t1 & MAY_BE_OBJECT) || (t2 & MAY_BE_OBJECT);

0 commit comments

Comments
 (0)