Bug #66327 [NEW]: Feedback on New 1.5 optimizations

From: Terry at ellisons dot org dot uk Operating system: N/A PHP version: master-Git-2013-12-20 (Git) Package: opcache Bug Type: Bug Bug description:Feedback on New 1.5 optimizations Description: ------------ Xinchen, You might be interested in the referenced PHPT which demonstrates how we can test the new optimizations introduced in Pass 1.5, which is the real point of this bugrep. I've gone through the function related ones and these seen to be fine, though it's a shame that we can't broaden the scope of these within B/C behaviours. One minor Q - the optimizations don't include class_exists() optimization for builtin classes and ditto is_callable() doesn't support builtin method probs such as "Reflection::export". Is there any reason for this? If we are happy with then please update line 82 of the test. I thought that I could create fail paths if the enable_dl directive is set with cached compiled scripts, but on reflection as this is disabled by all of the SAPIs which OPcache caches, this isn't an issue. If this isn't of interest then just close the rep. //Terry Test script: --------------- https://gist.github.com/TerryE/8048948 -- Edit bug report at https://bugs.php.net/bug.php?id=66327&edit=1 -- Try a snapshot (PHP 5.4): https://bugs.php.net/fix.php?id=66327&r=trysnapshot54 Try a snapshot (PHP 5.5): https://bugs.php.net/fix.php?id=66327&r=trysnapshot55 Try a snapshot (trunk): https://bugs.php.net/fix.php?id=66327&r=trysnapshottrunk Fixed in SVN: https://bugs.php.net/fix.php?id=66327&r=fixed Fixed in release: https://bugs.php.net/fix.php?id=66327&r=alreadyfixed Need backtrace: https://bugs.php.net/fix.php?id=66327&r=needtrace Need Reproduce Script: https://bugs.php.net/fix.php?id=66327&r=needscript Try newer version: https://bugs.php.net/fix.php?id=66327&r=oldversion Not developer issue: https://bugs.php.net/fix.php?id=66327&r=support Expected behavior: https://bugs.php.net/fix.php?id=66327&r=notwrong Not enough info: https://bugs.php.net/fix.php?id=66327&r=notenoughinfo Submitted twice: https://bugs.php.net/fix.php?id=66327&r=submittedtwice register_globals: https://bugs.php.net/fix.php?id=66327&r=globals PHP 4 support discontinued: https://bugs.php.net/fix.php?id=66327&r=php4 Daylight Savings: https://bugs.php.net/fix.php?id=66327&r=dst IIS Stability: https://bugs.php.net/fix.php?id=66327&r=isapi Install GNU Sed: https://bugs.php.net/fix.php?id=66327&r=gnused Floating point limitations: https://bugs.php.net/fix.php?id=66327&r=float No Zend Extensions: https://bugs.php.net/fix.php?id=66327&r=nozend MySQL Configuration Error: https://bugs.php.net/fix.php?id=66327&r=mysqlcfg