Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:65835 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 65708 invoked from network); 14 Feb 2013 16:02:21 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 14 Feb 2013 16:02:21 -0000 Authentication-Results: pb1.pair.com smtp.mail=nikita.ppv@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=nikita.ppv@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.219.43 as permitted sender) X-PHP-List-Original-Sender: nikita.ppv@gmail.com X-Host-Fingerprint: 209.85.219.43 mail-oa0-f43.google.com Received: from [209.85.219.43] ([209.85.219.43:48524] helo=mail-oa0-f43.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 14/61-58622-C8A0D115 for ; Thu, 14 Feb 2013 11:02:21 -0500 Received: by mail-oa0-f43.google.com with SMTP id l10so2684823oag.2 for ; Thu, 14 Feb 2013 08:02:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:cc:content-type; bh=n5mFk7SPOrMqLGt+7i4dxcfn3uH9zG+pSWfrgjYQrXk=; b=WQl5ZNoC9mYO2rgA0W3XN3+b1+LuiL5modj5Nj/e3oJnYQzYIeWa5URkgT3glUpxbS uf0EZ7E9zgpOQt74OBcxp++1C9JXyj4SYzF9Y3Mqv9Hjs+u0/PK2zVRp6BSSyJMu2TNy QBuPFL63wY4qdabu6tHzgf6fyXgg5aFdv/eeeKY2I5i4El7U1yPVsO7Wg0UiZsz7Bx4x IHdCAa1tsDDMvt6fS2vek4GIyzxHSYfo59EWoXdlHdo12WPMi90K3GVvm19UwU3QKRl1 WtbVHVbNvkqZUITBqPT+CdY6jEGbGOJb7nD46PzEDie/+IvErrsWaUkXKtoqD0/8mjrd y6sQ== MIME-Version: 1.0 X-Received: by 10.182.161.39 with SMTP id xp7mr20358063obb.71.1360857738192; Thu, 14 Feb 2013 08:02:18 -0800 (PST) Received: by 10.182.49.136 with HTTP; Thu, 14 Feb 2013 08:02:17 -0800 (PST) In-Reply-To: <7de2703f77537a47b457c4479a19ac3a@mail.gmail.com> References: <511BFC81.8020400@oracle.com> <7de2703f77537a47b457c4479a19ac3a@mail.gmail.com> Date: Thu, 14 Feb 2013 17:02:17 +0100 Message-ID: To: Zeev Suraski Cc: Christopher Jones , PHP internals Content-Type: multipart/alternative; boundary=f46d0442876813261304d5b162de Subject: Re: [PHP-DEV] Zend Optimizer+ Source Code now available From: nikita.ppv@gmail.com (Nikita Popov) --f46d0442876813261304d5b162de Content-Type: text/plain; charset=ISO-8859-1 On Thu, Feb 14, 2013 at 4:21 PM, Zeev Suraski wrote: > > - Should the name reflect the code's main purpose (op-code caching), > > and allowing a future use of "optimizer" for a more sophisticated > > optimizer implementation? Or do you see Optimizer+ being the > > framework for such optimizations? > > O+ does perform some optimizations in addition to caching code, in a pretty > sophisticated manner actually (block optimizations). Optimizations - which > can be expensive to carry out - are definitely a good fit with an opcode > cache, that ensures that you wouldn't have to do these optimizations more > than once. I'm obviously subjective but I think the name Optimizer+ does a > good job at suggesting that it's both an Optimizer but also something else. > Perhaps we should call it OptiCache? :) > If this will go into PECL first then I see no reason to change the name from Optimizer+. If this will go into PHP then it shouldn't need a name at all, should it? It could just be "opcode cache" (--enable-opcode-cache / --disable-opcode-cache). That seems more descriptive to me then some fancy name like "Optimizer+". Regarding the optimizations it contains, imho those are a separate concern and if Optimizer+ goes into core both aspects should be cleanly separated (and you should be able to enable/disable them separately). The optimizations are not directly related to caching. Caching makes them more viable for web requests, but as someone already pointed out the optimizations are also useful on CLI, where compile times just aren't a concern anyway (but run times can be). Btw, I was quite surprised to see the block optimizations in O+ :) Really cool! Nikita --f46d0442876813261304d5b162de--