Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:26448
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 212.25.124.162 as permitted sender)
To: "'Nuno Lopes'" <nlopess@php.net>,
	"'PHPdev'" <internals@lists.php.net>
Date: Thu, 9 Nov 2006 19:33:21 -0800
Message-ID: <001801c70478$fa9bf560$6800a8c0@zend.2k>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
In-Reply-To: <000701c70421$bae65e80$0100a8c0@pc07653>
Thread-Index: AccEIZ+DGMmQNgpjTUKWhq7CCyaVJAAV00WQ
Subject: RE: [PHP-DEV] pushing some more optimization patches
From: andi@zend.com ("Andi Gutmans")
References: <000701c70421$bae65e80$0100a8c0@pc07653>

The arginfo patch looks OK to me. Not sure about the visibility patch. I'll
try and look into it tomorrow. 

> -----Original Message-----
> From: Nuno Lopes [mailto:nlopess@php.net] 
> Sent: Thursday, November 09, 2006 9:09 AM
> To: PHPdev
> Subject: [PHP-DEV] pushing some more optimization patches
> 
> Hi,
> 
> 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.
> 
>  * 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.
> 
> Any opinions or any volunteer to commit those patches? :)
> 
> Thanks,
> Nuno 
> 
> --
> PHP Internals - PHP Runtime Development Mailing List To 
> unsubscribe, visit: http://www.php.net/unsub.php
>