Req #76241 [Opn->Fbk]: Add hrtime() to zend_try_compile_special_func()
| From: | ab@php.net | Date: | Tue, 19 Jun 2018 20:44:09 +0000 |
| Subject: | Req #76241 [Opn->Fbk]: Add hrtime() to zend_try_compile_special_func() | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-215801@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76241&edit=1
ID: 76241
Updated by: ab@php.net
Reported by: nicolas dot grekas+php at gmail dot com
Summary: Add hrtime() to zend_try_compile_special_func()
-Status: Open
+Status: Feedback
Type: Feature/Change Request
Package: Scripting Engine problem
PHP Version: Next Minor Version
Block user comment: N
Private report: N
New Comment:
Thanks for filing the issue. On the one hand, the idea makes a great sense, as if implemented it
would allow for more precise timing. On the other hand, the tight integration into the VM would mean
one more case clause, whereby the function itself wouldn't be expected to be used very often.
Before we strive that, perhaps it would be good to see some justified need. Otherwise, given there
might be some lapse of a couple of nanoseconds, the results should be still acceptable for most of
usages.
Thanks.
Previous Comments:
------------------------------------------------------------------------
[2018-04-19 09:56:06] nicolas dot grekas+php at gmail dot com
Description:
------------
hrtime() is coming to PHP 7.3 for high resolution timing measurements.
It would be great to add to zend_try_compile_special_func() in zend_compile.c so that the overhead
of the function call is removed from measurements.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=76241&edit=1