> On Dec. 8, 2014, 2:36 p.m., Andreas Hansson wrote:
> > src/cpu/kvm/vm.cc, line 374
> > <http://reviews.gem5.org/r/2510/diff/2/?file=42734#file42734line374>
> >
> >     I think this causes problems with some of the officially supported 
> > compilers. It's just a hunch, but please check.
> 
> Gabe Black wrote:
>     Ugh. Yes, I think you're right. That's standard C but not C++.

I changed it to just initialize the elements one by one.


- Gabe


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


On Nov. 23, 2014, 2:51 p.m., Gabe Black wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/2510/
> -----------------------------------------------------------
> 
> (Updated Nov. 23, 2014, 2:51 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 10551:7767dc21318d
> ---------------------------
> Let other objects set up memory like regions in a KVM VM.
> 
> 
> Diffs
> -----
> 
>   src/cpu/kvm/vm.hh f9fb64a72259a2514080151b5250a04c575d443a 
>   src/cpu/kvm/vm.cc f9fb64a72259a2514080151b5250a04c575d443a 
> 
> Diff: http://reviews.gem5.org/r/2510/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gabe Black
> 
>

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

Reply via email to