Skip to content

Commit b88d387

Browse files
committed
Revert "Fix incorrect trace type inference in a false loop"
This reverts commit 7f06a9d.
1 parent 7f06a9d commit b88d387

File tree

2 files changed

+2
-38
lines changed

2 files changed

+2
-38
lines changed

ext/opcache/jit/zend_jit_trace.c

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -787,19 +787,6 @@ static int zend_jit_trace_add_ret_phis(zend_jit_trace_rec *trace_buffer, uint32_
787787
return ssa_vars_count;
788788
}
789789

790-
static bool zend_jit_trace_is_false_loop(const zend_op_array *op_array, const zend_ssa *ssa, const zend_op **tssa_opcodes, zend_ssa *tssa)
791-
{
792-
uint32_t b;
793-
zend_basic_block *bb;
794-
795-
ZEND_ASSERT(tssa->cfg.blocks_count == 2);
796-
ZEND_ASSERT(tssa->cfg.blocks[1].len > 0);
797-
798-
b = ssa->cfg.map[tssa_opcodes[0] - op_array->opcodes];
799-
bb = ssa->cfg.blocks + ssa->cfg.map[tssa_opcodes[tssa->cfg.blocks[1].len - 1] - op_array->opcodes];
800-
return bb->loop_header != b;
801-
}
802-
803790
static int zend_jit_trace_copy_ssa_var_info(const zend_op_array *op_array, const zend_ssa *ssa, const zend_op **tssa_opcodes, zend_ssa *tssa, int ssa_var)
804791
{
805792
int var, use, def, src;
@@ -809,8 +796,7 @@ static int zend_jit_trace_copy_ssa_var_info(const zend_op_array *op_array, const
809796
uint32_t b = ssa->cfg.map[tssa_opcodes[0] - op_array->opcodes];
810797
zend_basic_block *bb = ssa->cfg.blocks + b;
811798

812-
if ((bb->flags & ZEND_BB_LOOP_HEADER)
813-
&& !zend_jit_trace_is_false_loop(op_array, ssa, tssa_opcodes, tssa)) {
799+
if (bb->flags & ZEND_BB_LOOP_HEADER) {
814800
zend_ssa_phi *phi = ssa->blocks[b].phis;
815801
zend_ssa_phi *pi = NULL;
816802

@@ -1405,8 +1391,7 @@ static zend_ssa *zend_jit_trace_build_tssa(zend_jit_trace_rec *trace_buffer, uin
14051391
tssa->cfg.blocks[0].successors = tssa->cfg.blocks[0].successors_storage;
14061392
tssa->cfg.blocks[0].successors[0] = 1;
14071393

1408-
tssa->cfg.blocks[1].flags = ZEND_BB_FOLLOW|ZEND_BB_TARGET|ZEND_BB_LOOP_HEADER|ZEND_BB_REACHABLE;
1409-
tssa->cfg.blocks[1].len = ssa_ops_count;
1394+
tssa->cfg.blocks[0].flags = ZEND_BB_FOLLOW|ZEND_BB_TARGET|ZEND_BB_LOOP_HEADER|ZEND_BB_REACHABLE;
14101395
tssa->cfg.blocks[1].successors_count = 1;
14111396
tssa->cfg.blocks[1].predecessors_count = 2;
14121397
tssa->cfg.blocks[1].successors = tssa->cfg.blocks[1].successors_storage;
@@ -1416,7 +1401,6 @@ static zend_ssa *zend_jit_trace_build_tssa(zend_jit_trace_rec *trace_buffer, uin
14161401
tssa->cfg.edges_count = 0;
14171402

14181403
tssa->cfg.blocks[0].flags = ZEND_BB_START|ZEND_BB_EXIT|ZEND_BB_REACHABLE;
1419-
tssa->cfg.blocks[0].len = ssa_ops_count;
14201404
tssa->cfg.blocks[0].successors_count = 0;
14211405
tssa->cfg.blocks[0].predecessors_count = 0;
14221406
}

ext/opcache/tests/jit/loop_004.phpt

Lines changed: 0 additions & 20 deletions
This file was deleted.

0 commit comments

Comments
 (0)