Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2756
to look at the new patch set (#2).
Change subject: Add NumFunctions and NumInstructions counters to LlvmCodeGen
......................................................................
Add NumFunctions and NumInstructions counters to LlvmCodeGen
These counters give a rough idea of how much "work" the module
optimization and compilation need to do, and are especially useful
when reasoning about the cost (or benefit) of new codegen
changes. This change doesn't have any measurable impact on codegen
time for a reasonably-sized module (372 functions, ~5k instructions).
Change-Id: I9a7f4aa02239d0738d891b09fa4531b560428898
---
M be/src/codegen/llvm-codegen.cc
M be/src/codegen/llvm-codegen.h
2 files changed, 14 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/56/2756/2
--
To view, visit http://gerrit.cloudera.org:8080/2756
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9a7f4aa02239d0738d891b09fa4531b560428898
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Skye Wanderman-Milne <[email protected]>
Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
