> On Dec. 4, 2014, 4:01 p.m., Steve Reinhardt wrote:
> > src/base/remote_gdb.hh, line 157
> > <http://reviews.gem5.org/r/2538/diff/1/?file=42808#file42808line157>
> >
> >     a comment here like '// size of cache in bytes' would be nice just to 
> > be clear (and since that has changed)

Fixed.


- Gabe


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


On Nov. 25, 2014, 11:47 a.m., Gabe Black wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/2538/
> -----------------------------------------------------------
> 
> (Updated Nov. 25, 2014, 11:47 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 10566:b350c8558d2f
> ---------------------------
> misc: Make the GDB register cache accessible in various sized chunks.
> 
> Not all ISAs have 64 bit sized registers, so it's not always very convenient
> to access the GDB register cache in 64 bit sized chunks. This change makes it
> accessible in 8, 16, 32, or 64 bit chunks. The MIPS and ARM implementations
> were working around that limitation by bundling and unbundling 32 bit values
> into 64 bit values. That code has been removed.
> 
> 
> Diffs
> -----
> 
>   src/arch/alpha/remote_gdb.cc f9fb64a72259a2514080151b5250a04c575d443a 
>   src/arch/arm/remote_gdb.hh f9fb64a72259a2514080151b5250a04c575d443a 
>   src/arch/arm/remote_gdb.cc f9fb64a72259a2514080151b5250a04c575d443a 
>   src/arch/mips/remote_gdb.hh f9fb64a72259a2514080151b5250a04c575d443a 
>   src/arch/mips/remote_gdb.cc f9fb64a72259a2514080151b5250a04c575d443a 
>   src/arch/sparc/remote_gdb.cc f9fb64a72259a2514080151b5250a04c575d443a 
>   src/base/remote_gdb.hh f9fb64a72259a2514080151b5250a04c575d443a 
> 
> Diff: http://reviews.gem5.org/r/2538/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gabe Black
> 
>

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

Reply via email to