Bug #66327 [Asn->Csd]: Feedback on New 1.5 optimizations
| From: | Terry at ellisons dot org dot uk | Date: | Fri, 10 Jan 2014 23:44:36 +0000 |
| Subject: | Bug #66327 [Asn->Csd]: Feedback on New 1.5 optimizations | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-183707@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=66327&edit=1
ID: 66327
User updated by: Terry at ellisons dot org dot uk
Reported by: Terry at ellisons dot org dot uk
Summary: Feedback on New 1.5 optimizations
-Status: Assigned
+Status: Closed
Type: Bug
Package: opcache
Operating System: N/A
PHP Version: master-Git-2013-12-20 (Git)
Assigned To: laruence
Block user comment: N
Private report: N
New Comment:
Closed by O/P
Previous Comments:
------------------------------------------------------------------------
[2013-12-20 01:31:15] Terry at ellisons dot org dot uk
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 this bug report at https://bugs.php.net/bug.php?id=66327&edit=1