[GitHub] [arrow] jianxind commented on pull request #7607: ARROW-8996: [C++] Add AVX version for aggregate sum/mean with runtime dispatch

2020-07-08 Thread GitBox
jianxind commented on pull request #7607: URL: https://github.com/apache/arrow/pull/7607#issuecomment-655383817 > @ursabot benchmark --suite-filter=arrow-compute-aggregate-benchmark The result of the buildbot, https://ci.ursalabs.org/#/builders/73/builds/93. The clang compiler can

[GitHub] [arrow] jianxind commented on pull request #7607: ARROW-8996: [C++] Add AVX version for aggregate sum/mean with runtime dispatch

2020-07-08 Thread GitBox
jianxind commented on pull request #7607: URL: https://github.com/apache/arrow/pull/7607#issuecomment-655370919 @wesm @emkornfield @pitrou Ping, could you help to review this approach when you are available? Appreciate your help.

[GitHub] [arrow] jianxind commented on pull request #7607: ARROW-8996: [C++] Add AVX version for aggregate sum/mean with runtime dispatch

2020-07-08 Thread GitBox
jianxind commented on pull request #7607: URL: https://github.com/apache/arrow/pull/7607#issuecomment-655371150 @ursabot benchmark --suite-filter=arrow-compute-aggregate-benchmark This is an automated message from the Apache

[GitHub] [arrow] jianxind commented on pull request #7607: ARROW-8996: [C++] Add AVX version for aggregate sum/mean

2020-07-01 Thread GitBox
jianxind commented on pull request #7607: URL: https://github.com/apache/arrow/pull/7607#issuecomment-652431931 @emkornfield This is the new version for sum aggregate without intrinsic, could you help to review? The dense part nearly get the same scores with intrinsic for AVX2 on

[GitHub] [arrow] jianxind commented on pull request #7607: ARROW-8996: [C++] Add AVX version for aggregate sum/mean

2020-07-01 Thread GitBox
jianxind commented on pull request #7607: URL: https://github.com/apache/arrow/pull/7607#issuecomment-652425725 @ursabot benchmark --suite-filter=arrow-compute-aggregate-benchmark This is an automated message from the Apache