develooper Front page | perl.perl5.porters | Postings from September 2016

Re: [PATCH] Speed up compilation of overload.pm a smidge

Thread Previous
From:
Karl Williamson
Date:
September 28, 2016 15:10
Subject:
Re: [PATCH] Speed up compilation of overload.pm a smidge
Message ID:
6a7023fd-7696-b8c6-49b4-87a4069f2c61@khwilliamson.com
On 09/28/2016 08:51 AM, James Raspass wrote:
> Thanks! I'd completely forgotten about this.
>
> I understand the change wasn't large/important but if I don't want to
> fall through the cracks in future is it better to open a bug before
> mailing in a out-of-the-blue patch?

Yes.  Include the patch in the RT ticket
>
> On 28 September 2016 at 09:49, Dave Mitchell <davem@iabyn.com
> <mailto:davem@iabyn.com>> wrote:
>
>     On Thu, Jul 16, 2015 at 12:00:06AM +0100, James Raspass wrote:
>     > Measured with the following crude perl script calling perf. Perl
>     > is in there to get a rough baseline cost of starting perl:
>     >
>     >  print 'PERL', (`perf stat -r100 perl -e 1             2>&1`)[10];
>     >  print 'OLD ', (`perf stat -r100 perl lib/overload.pm
>     <http://overload.pm>  2>&1`)[10];
>     >  print 'NEW ', (`perf stat -r100 perl lib/overload2.pm
>     <http://overload2.pm> 2>&1`)[10];
>     >
>     > Produced the following results on my machine:
>     >
>     >  PERL  5,800,051 instructions # 1.05 insns per cycle ( +- 0.06% )
>     >  OLD  14,818,995 instructions # 1.16 insns per cycle ( +- 0.03% )
>     >  NEW  14,696,974 instructions # 1.16 insns per cycle ( +- 0.03% )
>     >
>     > While the numbers did fluctuate between runs, the new code was
>     > consistently faster.
>
>     Thanks, applied as v5.25.5-43-g607ee43.
>
>     --
>     That he said that that that that is is is debatable, is debatable.
>
>


Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About