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



src/cpu/o3/fetch.hh
<http://reviews.m5sim.org/r/340/#comment925>

    done



src/cpu/o3/fetch.hh
<http://reviews.m5sim.org/r/340/#comment926>

    done



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/340/#comment928>

    It's not gone, it's moved... Need to check that the physical address still 
is what we think it is after translation. 



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/340/#comment927>

    it's really dated.. done



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/340/#comment931>

    you're right buildInst should have been used and then it will be ~ the same 
size



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/340/#comment929>

    done



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/340/#comment930>

    yes


- Ali


On 2010-12-06 16:11:45, Ali Saidi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/340/
> -----------------------------------------------------------
> 
> (Updated 2010-12-06 16:11:45)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> O3: Support timing translations for O3 CPU fetch.
> 
> 
> Diffs
> -----
> 
>   src/cpu/o3/fetch.hh 2b5fbdcbfb5d 
>   src/cpu/o3/fetch_impl.hh 2b5fbdcbfb5d 
> 
> Diff: http://reviews.m5sim.org/r/340/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ali
> 
>

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

Reply via email to