Internal Jenkins has submitted this change and it was merged. Change subject: Add FNV, Zobrist, and SIMD hash functions to the int hash benchmark. ......................................................................
Add FNV, Zobrist, and SIMD hash functions to the int hash benchmark. Additionally, change the parameter of rotate to a compile-time constant, and add "inline" to functions, increasing the performance dramatically. The compiler can't inline the SIMD versions, because they use Intel intrinsics -- added a TODO to add these intrinsics to sse-util.h. Change-Id: I11d48f8816d5b129858a1f773015e51049dd1d61 Reviewed-on: http://gerrit.cloudera.org:8080/4313 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Internal Jenkins --- M be/src/benchmarks/int-hash-benchmark.cc 1 file changed, 245 insertions(+), 57 deletions(-) Approvals: Internal Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4313 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11d48f8816d5b129858a1f773015e51049dd1d61 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
