Bug #76029 [Com]: Serious regression in foreach() looping
| From: | mark dot scherer at gmx dot de | Date: | Thu, 01 Mar 2018 13:50:53 +0000 |
| Subject: | Bug #76029 [Com]: Serious regression in foreach() looping | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-214165@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76029&edit=1
ID: 76029
Comment by: mark dot scherer at gmx dot de
Reported by: mark dot scherer at spryker dot com
Summary: Serious regression in foreach() looping
Status: Feedback
Type: Bug
Package: *General Issues
Operating System: Linux
PHP Version: 7.2.2
Block user comment: N
Private report: N
New Comment:
Yes, using optimization_level=0 also works fine.
Thank you!
Previous Comments:
------------------------------------------------------------------------
[2018-02-28 22:01:24] nikic@php.net
Looking at the code, my best guess is that we're contracting the assignment for
$shouldBeTextArea into IS_SMALLER and then something eats the NOP, resulting in a smart branch. But
I can't reproduce this. Would be nice to have a self-contained reproducing script for this.
Instead of disabling opcache entirely, it should be enough to set opcache.optimization_level=0 until
this is fixed.
------------------------------------------------------------------------
[2018-02-28 17:19:56] mark dot scherer at gmx dot de
OK, so disabled opcache and all is fine, so this at least limits itself to the opcode cache. We will
keep it disabled for now - but this will sure kill a lot of php applications once deployed.
------------------------------------------------------------------------
[2018-02-28 17:14:44] mark dot scherer at gmx dot de
opcode is on, I will disable and try again
$value = ...;
was a missing line I removed before the "$shouldBeTextArea = mb_strlen($value) > 255;",
but it does not change the result/report.
------------------------------------------------------------------------
[2018-02-28 16:54:34] peehaa@php.net
Could you also test it with opcache enabled and disabled please and post the results?
------------------------------------------------------------------------
[2018-02-28 16:50:52] peehaa@php.net
No repro: https://3v4l.org/fuGjJ
Please provide a simplified and working repro case.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=76029
--
Edit this bug report at https://bugs.php.net/bug.php?id=76029&edit=1