Hi Bert,

On Jan 28, 2015, at 2:45 AM, Bert Freudenberg <bert at freudenbergs.de> wrote:

> On 27.01.2015, at 22:55, Eliot Miranda <eliot.miranda at gmail.com> wrote:
>> 
>> 
>>> 99.98 (5,018)  UndefinedObject  DoIt
>>>   99.98 (5,018)  BlockClosure  bench
>>>     99.96 (5,017)  UndefinedObject  DoIt
>>>       20.9 (1,049)  Array [SequenceableCollection]  do:
>>>         |16.01 (804)  UndefinedObject  DoIt
>>>         |  16.01 (803)  Array [SequenceableCollection]  findLast:
>>>       5.7 (286)  Time class  millisecondClockValue
>>> 
>>> Which is much better, and almost exact. #millisecondsClockValue is sent from #bench, not from a block.
>> 
>> There's a printing bug there.  IMO it should look like
>> 
>>  99.98 (5,018)  UndefinedObject  DoIt
>>   99.98 (5,018)  BlockClosure  bench
>>     99.96 (5,017)  [] in UndefinedObject  DoIt
>>       20.9 (1,049)  Array [SequenceableCollection]  do:
>>         |16.01 (804)  [] in UndefinedObject  DoIt
>>         |  16.01 (803)  Array [SequenceableCollection]  findLast:
>>       5.7 (286)  Time class  millisecondClockValue
>> 
>> I'll take a look.
> 
> 
> Tallies currently only count per-method, not per-block. Maybe that is the reason why previously it tried to go via a block’s home?

Perhaps.  I fixed it in Andreas' profiler by adding a block nesting count to ASPTally.  I'll back port it to MessageTally soon.

> 
> - Bert -
> 
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20150128/deb2c9dd/attachment.htm