Re: [PATCH] Executor improvements

Out of curiosity, what's the slowdown without fastcall? Despite the potential slowdown, I still think this is the way to go. The engine isn't supposed to crash! Nuno ----- Original Message ----- From: "Dmitry Stogov" <dmitry@zend.com> To: "PHP Internals List" <internals@lists.php.net> Cc: "Andi Gutmans" <andi@zend.com>; "Stanislav Malyshev" <stas@zend.com> Sent: Tuesday, June 10, 2008 12:11 PM Subject: [PHP-DEV] [PATCH] Executor improvements

Hi, The proposed patch for PHP improves the executor. At first it eliminates direct execute() recursion so the following script won't produce SIGSEGV anymore (It'll produce memory overflow error instead). <?php function foo() { foo(); } foo(); ?> Note that in case some extension (e.q. xdebug) override zend_execute(), VM will use recursive calls. At second executor now uses "fastcall" calling convention, and as result it gives ~5% speedup on bench.php (~3.8 sec instead of ~4.0 sec) Any objections? Thanks. Dmitry.