On 10/23/2017 05:34 PM, Lukas Mai wrote: > I think it would be a good idea to use compiler intrinsics for overflow > checks where available. > > I've pushed a branch that implements this at mauke/overflow: > https://perl5.git.perl.org/perl.git/shortlog/refs/heads/mauke/overflow > > Dave: I've CC'd you directly because you last worked on this code > (commit 230ee21f3e366901ce5769d324124c522df7ce8a, "faster add, subtract, > multiply"). > > My changes affect pp_add, pp_subtract, and pp_multiply. I think the new > code is nicer because it's easier to understand than all the low-level > bit fiddling, and it passes all tests on my machine. However, I haven't > done any benchmarks to see how it affects performance (if at all). > > Things I need help with: > > - code review > - testing on different platforms / architectures > - benchmarks (compared to a17768d7c7b82c136fbeacd85db3451973a8007a) > 1. You might want to rename your branch 'smoke-me/mauke/overflow'. Lacking the 'smoke-me/', it won't appear in the drop-down box at http://perl.develop-help.com/?b=blead (even though test-smoke.org will show it). 2. Please see these smoke-test reports on FreeBSD-10.3 and FreeBSD-11.0, respectively: 10.3: http://perl5.test-smoke.org/report/58864 I think the one test failure is spurious: a side effect of resource constraints on this VM on my laptop. 11.0: http://perl5.test-smoke.org/report/58869 Significant failures in t/op/numconvert.t. Attaching output. Thank you very much. Jim KeenanThread Previous | Thread Next