Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:79242
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.177 as permitted sender)
MIME-Version: 1.0
Date: Thu, 27 Nov 2014 21:27:31 +0400
Message-ID: <CA+9eiLuiLUoKRrR7_Mp2UmYKMpwqcUPQ+Kd8zixuLQ3TQv7CTg@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>, Xinchen Hui <xinchen.h@zend.com>, 
	Matteo Beccati <matteo@beccati.com>, PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7b3431a088cb6b0508da759c
Subject: EX(scope) removing
From: dmitry@zend.com (Dmitry Stogov)

--047d7b3431a088cb6b0508da759c
Content-Type: text/plain; charset=UTF-8

Hi,

I'm working on call/return sequence optimization. As part of this work I'm
minimizing the size of call frame (zend_execute_data) and number of
read/write operations on call/return.

Could you please take a look into the patch that removes EX(scope)
https://gist.github.com/dstogov/5ad50d582346385d587e

All phpt tests are passed, but I'm not completely sure about one line -
https://gist.github.com/dstogov/5ad50d582346385d587e#file-ex_scope-05-diff-L193

On the other hand I can't imagine what it may break.

Matteo, can you please run your test suites with this patch.

PS: I'm also going to:
- pack EX(num_args) into reserved space of EX(This)
- try to get rid of EX(frame_info)
- try to merge EX(called_scope) and EX(This). Only one of them matters.
- try to replace zval EX(This) with a pointer to zend_object EX(object)

Thanks. Dmitry.

--047d7b3431a088cb6b0508da759c--