Re: [boost] : [math] Enable move semantics for polynomials
Lakshay Garg writes:
Hello all
The polynomial class in boost.math does not support move sematics. I tried to implement them and found that they immediately lead to significant improvements in performance (benchmarking using google-benchmark).
I would like to get the changes merged into the main repository. I understand that boost accepts very high quality C++ code
Hmm. Here's a benchmark where Boost doesn't do well: https://github.com/thekvs/cpp-serializers . Brian Ebenezer Enterprises - In G-d we trust. http://webEbenezer.net
On 14 August 2017 at 09:23, Brian Wood via Boost
Lakshay Garg writes:
Hello all
The polynomial class in boost.math does not support move sematics. I tried to implement them and found that they immediately lead to significant improvements in performance (benchmarking using google-benchmark).
I would like to get the changes merged into the main repository. I understand that boost accepts very high quality C++ code
Hmm. Here's a benchmark where Boost doesn't do well:
Hey Brian, thanks for the link. I am not sure how this link relates to the problem with polynomials. Could you please elaborate :) Regards Lakshay
On Sun, Aug 13, 2017 at 11:11 PM, Lakshay Garg
On 14 August 2017 at 09:23, Brian Wood via Boost
wrote: Lakshay Garg writes:
Hello all
The polynomial class in boost.math does not support move sematics. I tried to implement them and found that they immediately lead to significant improvements in performance (benchmarking using google-benchmark).
I would like to get the changes merged into the main repository. I understand that boost accepts very high quality C++ code
Hmm. Here's a benchmark where Boost doesn't do well:
Hey Brian, thanks for the link. I am not sure how this link relates to the problem with polynomials. Could you please elaborate :)
My comments were in reference to your comments about the quality of the code. Sorry if that wasn't clear. Brian Ebenezer Enterprises http://webEbenezer.net
participants (2)
-
Brian Wood
-
Lakshay Garg