Re: [boost] : [math] Enable move semantics for polynomials

Subject: Re: [boost] : [math] Enable move semantics for polynomials
From: Brian Wood (woodbrian77_at_[hidden])
Date: 2017-08-16 03:30:14

On Sun, Aug 13, 2017 at 11:11 PM, Lakshay Garg <lakshayg373_at_[hidden]>
wrote:

> On 14 August 2017 at 09:23, Brian Wood via Boost <boost_at_[hidden]>
> 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:
>>
>> https://github.com/thekvs/cpp-serializers
>>
>>
> ​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