Skip to content

Commit 525cc9a

Browse files
committed
Optimizer: Use true / false instead of 1 / 0 for bool parameters (2)
Changes done with Coccinelle: @r1@ identifier F; identifier p; typedef bool; type T; parameter list [n1] PL1; parameter list [n2] PL2; @@ T F(PL1, bool p, PL2); @r2@ identifier r1.F; expression list [r1.n1] EL1; expression list [r1.n2] EL2; @@ F(EL1, ( - 1 + true | - 0 + false ) , EL2)
1 parent ed4537d commit 525cc9a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Zend/Optimizer/zend_ssa.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -240,21 +240,21 @@ static zend_always_inline void zend_ssa_rename_defs_of_instr(zend_ssa *ssa, zend
240240
/* Rename def to use if possible. Mark variable as not defined otherwise. */
241241
if (ssa_op->op1_def >= 0) {
242242
if (ssa_op->op1_use >= 0) {
243-
zend_ssa_rename_var_uses(ssa, ssa_op->op1_def, ssa_op->op1_use, 1);
243+
zend_ssa_rename_var_uses(ssa, ssa_op->op1_def, ssa_op->op1_use, true);
244244
}
245245
ssa->vars[ssa_op->op1_def].definition = -1;
246246
ssa_op->op1_def = -1;
247247
}
248248
if (ssa_op->op2_def >= 0) {
249249
if (ssa_op->op2_use >= 0) {
250-
zend_ssa_rename_var_uses(ssa, ssa_op->op2_def, ssa_op->op2_use, 1);
250+
zend_ssa_rename_var_uses(ssa, ssa_op->op2_def, ssa_op->op2_use, true);
251251
}
252252
ssa->vars[ssa_op->op2_def].definition = -1;
253253
ssa_op->op2_def = -1;
254254
}
255255
if (ssa_op->result_def >= 0) {
256256
if (ssa_op->result_use >= 0) {
257-
zend_ssa_rename_var_uses(ssa, ssa_op->result_def, ssa_op->result_use, 1);
257+
zend_ssa_rename_var_uses(ssa, ssa_op->result_def, ssa_op->result_use, true);
258258
}
259259
ssa->vars[ssa_op->result_def].definition = -1;
260260
ssa_op->result_def = -1;

0 commit comments

Comments
 (0)