Bug #76029 [Com]: Serious regression in foreach() looping

From: 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

« previous php.bugs (#214165) next »