Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:27669 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 28302 invoked by uid 1010); 26 Jan 2007 14:38:08 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 28287 invoked from network); 26 Jan 2007 14:38:08 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 26 Jan 2007 14:38:08 -0000 Authentication-Results: pb1.pair.com header.from=pierre.php@gmail.com; sender-id=pass; domainkeys=bad Authentication-Results: pb1.pair.com smtp.mail=pierre.php@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 64.233.162.226 as permitted sender) DomainKey-Status: bad X-DomainKeys: Ecelerity dk_validate implementing draft-delany-domainkeys-base-01 X-PHP-List-Original-Sender: pierre.php@gmail.com X-Host-Fingerprint: 64.233.162.226 nz-out-0506.google.com Linux 2.4/2.6 Received: from [64.233.162.226] ([64.233.162.226:38604] helo=nz-out-0506.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id D0/83-45640-0521AB54 for ; Fri, 26 Jan 2007 09:38:08 -0500 Received: by nz-out-0506.google.com with SMTP id k1so788528nzf for ; Fri, 26 Jan 2007 06:38:06 -0800 (PST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=AITzjrHIA0YC/MyItKeiV5IT7n3MtUPPxvsTQ7Xt4ZcsYnkRh9K8wXvddBBmeQhSCPptznESw1quV3t/8Mvt/fj4o5FnsNEeLNq7oBo8zE7NVuZGwrQ7X3vGJkZ7xboT8B4loG+ZCXX30VM1SmHtP+fAovrLxD1ioEmSTScJHFM= Received: by 10.64.143.12 with SMTP id q12mr5028237qbd.1169822277879; Fri, 26 Jan 2007 06:37:57 -0800 (PST) Received: by 10.65.185.15 with HTTP; Fri, 26 Jan 2007 06:37:57 -0800 (PST) Message-ID: Date: Fri, 26 Jan 2007 15:37:57 +0100 To: "Dmitry Stogov" Cc: "Sara Golemon" , internals@lists.php.net, "Andi Gutmans" , "Andrei Zmievski" , "Zeev Suraski" , "Stanislav Malyshev" In-Reply-To: <000001c74156$c1b49a20$6e02a8c0@thinkpad> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <45B7BA56.40406@php.net> <000001c74156$c1b49a20$6e02a8c0@thinkpad> Subject: Re: [PHP-DEV] Runtime-JIT, the whole enchilada From: pierre.php@gmail.com (Pierre) Hi Dmitry, On 1/26/07, Dmitry Stogov wrote: > Hi Sara, > > I fixed one error in your patch, so now it passes all tests. > > BTW I am against this patch. > > 1) It adds sloppily built functionality into the core and slowdowns EACH > dimension or property fetch both at run-time and compilation-time. I think > it is a big cost for $_GET/$_POST encoding-conversion, but it is only my > opinion. > > 2) It adds pointer to zend_auto_global into zend_compiled_variable. This > makes troubles for opcode caches - zend_compiled_variable(s) may lay in > shared memory, so they cannot point to process specific data. I think this > issue can be fixed using indirect pointer (index). > > I would prefer rollback "auto-globals CV" patch and commit (or not) the > whole patch after conclusion. > > Why did you reject auto-globals as overloaded arrays? As I already told you in my initial proposal (which is simpler than this solution), they do not work in all cases (check the various bugs about them) and they will bring a huge breakage as is_array($objarray) returns false. The later is certainly fixable (with slowness too), I'm not sure about the other issues. --Pierre