Skip to content

expression: implement vectorized evaluation for builtinAcosSig#12221

Merged
sre-bot merged 2 commits intopingcap:masterfrom
jacklightChen:acos
Sep 17, 2019
Merged

expression: implement vectorized evaluation for builtinAcosSig#12221
sre-bot merged 2 commits intopingcap:masterfrom
jacklightChen:acos

Conversation

@jacklightChen
Copy link
Contributor

What problem does this PR solve?

implement vectorized evaluation for builtinAcosSig, for #12105

What is changed and how it works?

according to benchmark, about 2 times faster than before:

BenchmarkVectorizedBuiltinMathFunc/builtinAcosSig-VecBuiltinFunc-8                100000             20019 ns/op            0 B/op          0 allocs/op
BenchmarkVectorizedBuiltinMathFunc/builtinAcosSig-NonVecBuiltinFunc-8              50000             34863 ns/op            0 B/op          0 allocs/op

Check List

Tests

  • Unit test

@sre-bot sre-bot added the contribution This PR is from a community contributor. label Sep 17, 2019
@codecov
Copy link

codecov bot commented Sep 17, 2019

Codecov Report

Merging #12221 into master will decrease coverage by 0.0171%.
The diff coverage is 50%.

@@               Coverage Diff               @@
##             master    #12221        +/-   ##
===============================================
- Coverage   81.1842%   81.167%   -0.0172%     
===============================================
  Files           454       454                
  Lines         99066     98110       -956     
===============================================
- Hits          80426     79633       -793     
+ Misses        12875     12741       -134     
+ Partials       5765      5736        -29

Copy link
Contributor

@Reminiscent Reminiscent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Reminiscent Reminiscent added the status/LGT1 Indicates that a PR has LGTM 1. label Sep 17, 2019
Copy link
Contributor

@qw4990 qw4990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qw4990 qw4990 added status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 17, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Sep 17, 2019

/run-all-tests

@sre-bot sre-bot merged commit 2195147 into pingcap:master Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/expression contribution This PR is from a community contributor. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants