-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perf: avoid copying on semigroup operations #379
perf: avoid copying on semigroup operations #379
Conversation
92e19cc
to
250cb3e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change 4 commits:
Avoid copying twice on …
->
Avoid copying twice in …
Can the commit 4db1ac3 be moved to before “avoid copying twice on addition” for continuity?
a3b6bb3
refac(math): refactor fast retrun case of polynomial operations
->
refac(math): refactor fast return case of polynomial operations
250cb3e
to
37933a1
Compare
b73c402
to
58633c0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
tachyon/math/polynomials/univariate/univariate_evaluations_ops.h
Outdated
Show resolved
Hide resolved
58633c0
to
eea10b9
Compare
eea10b9
to
38ba29f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR avoids copying on semigroup operations, which are
Add()
,Mul()
,Double()
andSquare()
.