impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Armstrong (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) Reduce size of cross-compiled IR
Date Mon, 07 Mar 2016 18:38:55 GMT
Hello Alex Behm,

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

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:// refs/changes/64/2464/2
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iea04ad2e4b365564ee71082657262485d3a85446
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Marcel Kornacker <>
Gerrit-Reviewer: Tim Armstrong <>

View raw message