Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4381: Incorrect AVX version of BloomFilter::Or ......................................................................
IMPALA-4381: Incorrect AVX version of BloomFilter::Or The iteration on the loop incrementing the loop variable by the wrong amount. This version is hopefully clearer. I have tested it on an AVX_enabled machine and I have inspected the assembly. Change-Id: I18480c598fbf6b842398581acde6fc719c40ce27 Reviewed-on: http://gerrit.cloudera.org:8080/4866 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Internal Jenkins --- M be/src/benchmarks/bloom-filter-benchmark.cc M be/src/util/bloom-filter-test.cc M be/src/util/bloom-filter.cc 3 files changed, 136 insertions(+), 123 deletions(-) Approvals: Internal Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4866 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18480c598fbf6b842398581acde6fc719c40ce27 Gerrit-PatchSet: 5 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]>
