Mailing-List: contact dev-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@impala.incubator.apache.org
Message-Id: <201603071838.u27IcuNm014113@ip-10-146-233-104.ec2.internal>
Date: Mon, 7 Mar 2016 18:38:55 +0000
From: "Tim Armstrong (Code Review)" <gerrit@cloudera.org>
To: Alex Behm <alex.behm@cloudera.com>, impala-cr@cloudera.com,
        dev@impala.incubator.apache.org
CC: Marcel Kornacker <marcel@cloudera.com>, Dan Hecht <dhecht@cloudera.com>
Reply-To: tarmstrong@cloudera.com
Subject: 
 =?UTF-8?Q?[Impala-CR](cdh5-trunk)_Reduce_size_of_cross-compiled_IR=0A?=
In-Reply-To: 
 <gerrit.1457131991000.Iea04ad2e4b365564ee71082657262485d3a85446@gerrit.cloudera.org>
References: 
 <gerrit.1457131991000.Iea04ad2e4b365564ee71082657262485d3a85446@gerrit.cloudera.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.10-rc0

Hello Alex Behm,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/2464

to look at the new patch set (#2).

Change subject: Reduce size of cross-compiled IR
......................................................................

Reduce size of cross-compiled IR

Disable the instnamer pass to reduce bitcode file size. Enable
inlining with a low threshold to inline many trivial functions in
the cross-compiled IR (e.g. scoped_ptr accessor functions).
Add additional IR_ALWAYS_INLINE annotations to trivial functions
that are called on hot paths.

This means that the functions will be inlined at compile time and
codegen has to do less work at runtime.

The IR module size is reduced from ~1.9MB to ~1.4MB with this
change and codegen time is slightly reduced. The number of functions is
reduced from 5698 to 3883.

Change-Id: Iea04ad2e4b365564ee71082657262485d3a85446
---
M be/CMakeLists.txt
M be/src/codegen/CMakeLists.txt
M be/src/common/compiler-util.h
M be/src/common/status.h
M be/src/exec/hash-table.h
M be/src/exec/partitioned-hash-join-node.h
M be/src/runtime/row-batch.h
M be/src/runtime/tuple-row.h
8 files changed, 59 insertions(+), 41 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/64/2464/2
-- 
To view, visit http://gerrit.cloudera.org:8080/2464
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iea04ad2e4b365564ee71082657262485d3a85446
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <marcel@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>