Skip to content

expression: implement vectorized evaluation for builtinAbsUIntSig#12400

Merged
sre-bot merged 6 commits intopingcap:masterfrom
tangwz:fix_builtinAbsUIntSig
Sep 27, 2019
Merged

expression: implement vectorized evaluation for builtinAbsUIntSig#12400
sre-bot merged 6 commits intopingcap:masterfrom
tangwz:fix_builtinAbsUIntSig

Conversation

@tangwz
Copy link
Contributor

@tangwz tangwz commented Sep 25, 2019

What problem does this PR solve?

Implement vectorized evaluation for builtinAbsUIntSig.
Issue: #12103

What is changed and how it works?

BenchmarkVectorizedBuiltinMathFunc/builtinAbsUIntSig-VecBuiltinFunc-4           20000000               104 ns/op               0 B/op          0 allocs/op
BenchmarkVectorizedBuiltinMathFunc/builtinAbsUIntSig-NonVecBuiltinFunc-4          100000             19013 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 25, 2019
@tangwz tangwz changed the title expression: implement vectorized evaluation for builtinAbsUIntSig expression: implement vectorized evaluation for builtinAbsUIntSig Sep 25, 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.

Almost LGTM, but please reorg the import packages.

@codecov
Copy link

codecov bot commented Sep 26, 2019

Codecov Report

Merging #12400 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #12400   +/-   ##
===========================================
  Coverage   79.8172%   79.8172%           
===========================================
  Files           462        462           
  Lines        102340     102340           
===========================================
  Hits          81685      81685           
  Misses        14787      14787           
  Partials       5868       5868

@tangwz tangwz force-pushed the fix_builtinAbsUIntSig branch from 0b4b11e to a04082d Compare September 26, 2019 13:58
Copy link
Contributor

@SunRunAway SunRunAway left a comment

Choose a reason for hiding this comment

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

LGTM

@SunRunAway SunRunAway added the status/LGT1 Indicates that a PR has LGTM 1. label Sep 26, 2019
@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 27, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Sep 27, 2019

/run-all-tests

@sre-bot sre-bot merged commit 988e8fa into pingcap:master Sep 27, 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.

5 participants