---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3757/
---
(Updated Jan. 26, 2017, 11:20 a.m.)
Review request for Default.
Repository: gem5
> On Jan. 20, 2017, 5:11 p.m., Tony Gutierrez wrote:
> > LGTM, nice implementation. I just have an ask for a few more comments.
> >
> > 1) A high-level description of the API, and some direction on usage for
> > this library would be useful as a top-level comment in the file.
> > 2) LaneData
> On Jan. 19, 2017, 10:28 p.m., Jason Lowe-Power wrote:
> > Seems OK to me. Don't let anyone ever tell you that C++ makes reading code
> > easy... I appreciate all of the compile-time tricks going on here, but it
> > is incredibly difficult to read.
> >
> > Maybe some more comments for each
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3757/#review9292
---
Ship it!
LGTM, nice implementation. I just have an ask for a few more
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3757/#review9286
---
Ship it!
Seems OK to me. Don't let anyone ever tell you that C++ makes
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3757/
---
(Updated Jan. 16, 2017, 11:54 a.m.)
Review request for Default.
Repository: gem5
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3757/
---
Review request for Default.
Repository: gem5
Description
---
Changeset
