Todd Lipcon has submitted this change and it was merged. Change subject: codegen: set JIT target to running host ......................................................................
codegen: set JIT target to running host This slight tweak to codegen tells the JIT engine to generate code for the currently running host. I looked at the generated assembly from codegen-test, and it's unchanged for integer projections. When I added a string column to the projection, this patch saved two instructions in the generated code by using some AVX instructions. Change-Id: I49c26696db0bfd819f918aec27ff5d65a55dcc93 Reviewed-on: http://gerrit.cloudera.org:8080/2653 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Todd Lipcon <[email protected]> --- M src/kudu/codegen/codegen-test.cc M src/kudu/codegen/module_builder.cc 2 files changed, 16 insertions(+), 1 deletion(-) Approvals: Adar Dembo: Looks good to me, approved Todd Lipcon: Verified -- To view, visit http://gerrit.cloudera.org:8080/2653 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49c26696db0bfd819f918aec27ff5d65a55dcc93 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
