Subject: Re: [boost] [math] Enable move semantics for polynomials
From: John Maddock (jz.maddock_at_[hidden])
Date: 2017-08-14 07:07:03
On 13/08/2017 22:05, Robert Ramey via Boost wrote:
> On 8/13/17 4:31 AM, Lakshay Garg via Boost wrote:
>> Hello all
>> The polynomial class in boost.math does not support move sematics. I
>> 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 and since this
>> will be my first time contributing to boost I am looking forward to
>> assistance from other experienced contributors.
>> Could someone please advise me on how to move forward and get the
> Sounds worthwhile. While you're at it, how about considering making
> all the functions constexpr ?
constexpr support would require the underlying container (std::vector)
to be constexpr as well, in fact since new (variable sized) storage is
required, I don't see how that would be possible at all..... you could
as someone suggested have a fixed storage polynomial class with
multiplication yielding a different type... any non-trivial manipulation
would lead to a complete explosion of template instantiations though.
--- This email has been checked for viruses by AVG. http://www.avg.com
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk