---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/612/
---
Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/612/#review1035
---
Looks good to me.
- Gabe
On 2011-03-30 08:58:56, Ali Saidi wrote:
This does interact with Korey's change a bit, so hopefully we don't end
up stepping on each other too much. Since somebody's going to have to
update a patch anyway, I'll look at whether that result stuff in the
dyninst can go away too.
Gabe
On 03/30/11 12:09, Gabe Black wrote:
They're used in Checker and Ozone (and orthogonally overloaded in
InOrder it looks like). Checker and Ozone really need to be either
updated and disposed of, but until that happens I suppose it doesn't
make sense to make it worse -and- complicate these other two patches by
getting rid of the
updated -or- disposed of. One is sufficient :-).
Gabe
On 03/30/11 13:13, Gabe Black wrote:
They're used in Checker and Ozone (and orthogonally overloaded in
InOrder it looks like). Checker and Ozone really need to be either
updated and disposed of, but until that happens I suppose it doesn't