Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re^2: Speed/Efficiency tweaks for a fannkuch benchmark script?

by Roy Johnson (Monsignor)
on Dec 02, 2005 at 16:00 UTC ( [id://513622]=note: print w/replies, xml ) Need Help??


in reply to Re: Speed/Efficiency tweaks for a fannkuch benchmark script?
in thread Speed/Efficiency tweaks for a fannkuch benchmark script?

I noticed that your fast permutation algorithm passes an arrayref, but then copies the values out. There's no particular savings there compared to passing the array. That and a few other micro-optimizations squeezed 210+% more speed out (if you don't print — the savings are probably less significant with printing turned on.) Code (update: fixed, reducing savings, as expected) follows.
{ my $level = 0; sub permute_new { local @_ = @_; my $index = $level; my $printing = $level == $#_; if ($printing) { do { #print "@_\n"; @_[$index-1, $index] = @_[$index, $index-1] if $index != 0; } while $index-- > 0; } else { do { ++$level; &permute_new; --$level; @_[$index-1, $index] = @_[$index, $index-1] if $index != 0; } while $index-- > 0; } } } sub permute_orig { my ($aref, $level) = (@_, 0); my ($index, $copy, $printing) = ($level, [@$aref], $level+1 == @$a +ref); do { if ($printing) { #print "@$copy\n"; } else { permute_orig($copy, 1+$level); } @$copy[$index-1, $index] = @$copy[$index, $index-1] if $index != 0; } while $index-- > 0; } my $to = shift; use Time::HiRes qw( gettimeofday tv_interval ); my @start_time = gettimeofday(); permute_orig([1..$to]); print "\n"; my @end_time = gettimeofday(); print tv_interval ( \@start_time, \@end_time )," elapsed seconds for o +riginal.\n\n"; @start_time = gettimeofday(); permute_new(1..$to); @end_time = gettimeofday(); print tv_interval ( \@start_time, \@end_time )," elapsed seconds for n +ew.\n\n";

Caution: Contents may have been coded under pressure.

Replies are listed 'Best First'.
Re^3: Speed/Efficiency tweaks for a fannkuch benchmark script?
by robin (Chaplain) on Dec 02, 2005 at 16:10 UTC
    Interesting, but broken. (Uncomment the print and you'll see what I mean!)

    You can fix it by adding

    local @_ = @_;
    to the beginning of the sub, but that presumably makes it slower again.

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://513622]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others sharing their wisdom with the Monastery: (6)
As of 2024-03-28 11:02 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found