-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1419/#review3470
-----------------------------------------------------------



src/mem/ruby/common/Address.cc
<http://reviews.gem5.org/r/1419/#comment3514>

    int64_t to be in line with the rest of gem5?



src/mem/ruby/common/TypeDefines.hh
<http://reviews.gem5.org/r/1419/#comment3515>

    Take the plunge and use src/base/types.hh, i.e. uint64_t and int64_t?


- Andreas Hansson


On Sept. 13, 2012, 7:58 a.m., Nilay Vaish wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/1419/
> -----------------------------------------------------------
> 
> (Updated Sept. 13, 2012, 7:58 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Description
> -------
> 
> Changeset 9225:f2888917f08e
> ---------------------------
> ruby: eliminate typedef integer_t
> 
> 
> Diffs
> -----
> 
>   src/mem/ruby/common/Address.cc be1c1059438b 
>   src/mem/ruby/common/TypeDefines.hh be1c1059438b 
>   src/mem/ruby/profiler/Profiler.hh be1c1059438b 
>   src/mem/ruby/profiler/Profiler.cc be1c1059438b 
> 
> Diff: http://reviews.gem5.org/r/1419/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nilay Vaish
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to