> On 2011-01-08 22:26:47, Nathan Binkert wrote:
> > src/base/statistics.hh, line 1462
> > <http://reviews.m5sim.org/r/357/diff/1/?file=5833#file5833line1462>
> >
> >     Why do you want this?  Generally, we don't expose the internals of 
> > stats and I'm reluctant to do so.

Since the average is being printed, I am fine with not having such a function.


- Nilay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/357/#review679
-----------------------------------------------------------


On 2010-12-21 11:15:24, Nathan Binkert wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/357/
> -----------------------------------------------------------
> 
> (Updated 2010-12-21 11:15:24)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> stats: Add a histogram statistic type
> 
> 
> Diffs
> -----
> 
>   src/base/statistics.hh 4a3bddd74f36 
>   src/base/statistics.cc 4a3bddd74f36 
>   src/base/stats/info.hh 4a3bddd74f36 
>   src/base/stats/text.cc 4a3bddd74f36 
>   src/unittest/stattest.cc 4a3bddd74f36 
> 
> Diff: http://reviews.m5sim.org/r/357/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nathan
> 
>

_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to