On Sun, 29 Aug 2010, Nicolas Cellier wrote:
2010/8/29 Nicolas Cellier nicolas.cellier.aka.nice@gmail.com:
2010/8/29 Levente Uzonyi leves@elte.hu:
snip...
Maybe http://bugs.squeak.org/view.php?id=7109 would serve a bit too...
Hm. This looks interresting, I guess some optimizations are already implemented:
| x | x := SmallInteger maxVal raisedTo: 100. #(-1600 -1597 1597 1600) collect: [ :shift | [ 1 to: 100000 do: [ :i | x bitShift: shift ] ] timeToRun ]. "CogVM ===> #(150 147 271 64)" "SquakVM ===> #(97 118 343 71)"
I regenerated COG, and I get:
Original COG #(281 292 621 275) #(284 290 604 271) COG + 7109 #(167 239 509 289) #(172 246 500 288) Oops, it was development build...
So I retried with deployment: Original COG #(116 114 219 65) #(116 113 219 65) COG + 7109 #(47 77 120 61) #(46 75 120 61)
So I think yes, the patch should better be integrated. It's only a small contributor of crypto timings, but it counts.
Cool. It's time to move this thread to the vm-dev list to get these patches into the vm trunk.
Levente
Nicolas
Levente
I should have a look at source then...