Internal Jenkins has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2756 Reviewed-by: Dan Hecht <[email protected]> Tested-by: Internal Jenkins --- M be/src/codegen/llvm-codegen.cc M be/src/codegen/llvm-codegen.h 2 files changed, 14 insertions(+), 0 deletions(-) Approvals: Internal Jenkins: Verified Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/2756 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a7f4aa02239d0738d891b09fa4531b560428898 Gerrit-PatchSet: 3 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Skye Wanderman-Milne <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
