Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:26436
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 64.233.184.226 as permitted sender)
DomainKey-Status: good
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:in-reply-to:references:mime-version:x-priority:content-type:message-id:cc:content-transfer-encoding:from:subject:date:to:x-mailer:sender;
        b=gaYhgOuF8Aedv3Bi/XvhLneoMczQ35STeysZHLjrxj4le4JB/y2K11hj8zxQGkJLV1o5MFY1HFmYekOXzP5aCMw7onN0v8Y4pnlVMgg6H8dvccv1e4ql3TKJaYhu2EfyyMZJ9IJWawXH4K8ZkU5+ZNgI0Zlmrocr6uhsiUwxrqA=
In-Reply-To: <000701c70421$bae65e80$0100a8c0@pc07653>
References: <000701c70421$bae65e80$0100a8c0@pc07653>
Mime-Version: 1.0 (Apple Message framework v752.3)
Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed
Message-ID: <4F6399D7-319A-4E8B-B4FB-A819E77AB068@prohost.org>
Cc: "PHPdev" <internals@lists.php.net>
Content-Transfer-Encoding: 7bit
Date: Thu, 9 Nov 2006 12:54:10 -0500
To: Nuno Lopes <nlopess@php.net>
Sender: Ilia Alshanetsky <iliaal@gmail.com>
Subject: Re: [PHP-DEV] pushing some more optimization patches
From: ilia@prohost.org (Ilia Alshanetsky)


On 9-Nov-06, at 12:08 PM, Nuno Lopes wrote:

> I would like to push 2 more optimization patches onto the PHP tree:
> * make arginfo const (and save memory): http://mega.ist.utl.pt/ 
> ~ncpl/zend_arginfo_const.txt (for php 5.2.1)
> Trivial patch, which is expecially good for multi-core systems  
> sharing the same cache.

The first part looks fine, and if no one objects I'll commit it next  
week.


> * use -fvisibility when using gcc 4: http://mega.ist.utl.pt/~ncpl/ 
> php_gcc4_visibility.txt (for php 5.3/6)
> this is a trickier patch and needs some tweaking in extensions  
> exporting their own API. The little hacks in the Apache SAPIs are  
> needed because older apache versions didn't know anything about gcc 4.

I'd prefer to hold off on adding to 5.2 series at this time, lets try  
it out on 6.0 and if it works fine there perhaps port it back once  
5.3 is on the table.


Ilia Alshanetsky