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 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 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 changes merged.
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. John. --- This email has been checked for viruses by AVG. http://www.avg.com