Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:64140 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 6701 invoked from network); 3 Dec 2012 17:00:01 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 3 Dec 2012 17:00:01 -0000 Authentication-Results: pb1.pair.com header.from=dmitry@zend.com; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=dmitry@zend.com; spf=unknown; sender-id=unknown Received-SPF: unknown (pb1.pair.com: domain zend.com does not designate 209.85.216.170 as permitted sender) X-PHP-List-Original-Sender: dmitry@zend.com X-Host-Fingerprint: 209.85.216.170 mail-qc0-f170.google.com Received: from [209.85.216.170] ([209.85.216.170:41927] helo=mail-qc0-f170.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 40/A4-03565-F8ADCB05 for ; Mon, 03 Dec 2012 12:00:00 -0500 Received: by mail-qc0-f170.google.com with SMTP id d42so1798487qca.29 for ; Mon, 03 Dec 2012 08:59:57 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-gm-message-state; bh=Q+vfK/IicxfxMtk6YRoxWgU5e5REhgwdNtLhBp+QTGg=; b=dYn7sPoUAbaPlYi+ye5b3MDixma3GsFBvPLGhqA37n80Yo/Nj5b+VzrOp60KFCYrRw 6kRxz/y00ydX/IdOL1bh9hukHLc+JrQYO0ZoDrfYaDJm1JUg4mzwxQ2tFJm9hAaS8KNY iAH+SqvYIikYePamd2RVoerccERhyfE6EQ+SfBBHGQiYNNZtR3dhFC+nGjYCNog+xlJr 87omqKxHuIcVG8DCINS5XsXNKRd02DjTm/ccyJXHtZhQGNuapDqri/s0f1BRJ5qbJeUE 3fKeEfGCH9cwBWugW6Vf2P6qFYPg4tSe7LqQ2u/a9qjof2RHOCRrIuJcL+I+se9XZuQJ snHw== MIME-Version: 1.0 Received: by 10.49.48.45 with SMTP id i13mr19676211qen.48.1354553997013; Mon, 03 Dec 2012 08:59:57 -0800 (PST) Received: by 10.49.74.133 with HTTP; Mon, 3 Dec 2012 08:59:56 -0800 (PST) In-Reply-To: References: <50B90DA4.606@lerdorf.com> Date: Mon, 3 Dec 2012 19:59:56 +0300 Message-ID: To: Nikita Popov Cc: Rasmus Lerdorf , Derick , julien pauli , Yoram Bar Haim , PHP Internals Content-Type: multipart/alternative; boundary=047d7b678964d2346204cff5ad39 X-Gm-Message-State: ALoCoQk5kpUTfMY2c32vBl6cgnxaVNxSNVDp9dqtcOa4nuqOVVjd64f5aEeaCl4elv5+wre35WS121ZkP9h2h+gdqb9mme2IWLFe0CH9XTchmdm4XgcQPP57AFY7/GBFYlCCD0OeLoUn Subject: Re: [PHP-DEV] execute_data->Ts removing From: dmitry@zend.com (Dmitry Stogov) --047d7b678964d2346204cff5ad39 Content-Type: text/plain; charset=UTF-8 Unfortunately, I didn't see visible performance gain on x86. :( It's near the measurement mistake. Probably it's because reading from L1 data cache is very cheap. Thanks. Dmitry. On Mon, Dec 3, 2012 at 7:33 PM, Nikita Popov wrote: > On Mon, Dec 3, 2012 at 10:35 AM, Dmitry Stogov wrote: > >> The new proposed patch: http://pastebin.com/pj5fQTfN >> >> Now both execute_data->Ts and execute_data->CVs are removed and >> corresponding temporary and compiled variables accessed using >> "execute_data" as the base pointer. Temporary variables allocate directly >> before the "execute_data" in reverse order and compiled variables right >> after. So they can be accessed without any additional computations. The >> patch reduces the number of executed instructions and number of memory >> reads (about 8% less). >> > > Did you also test how much these 8% less memory reads improve performance? > Would be interesting to know :) > > Nikita > --047d7b678964d2346204cff5ad39--