mem, misc: fix building issue with CommMonitor (unused variables)
Review Request #3801 - Created Feb. 3, 2017 and submitted
| Information | |
|---|---|
| Pierre-Yves Péneau | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 11808:e0730e1d7afa --------------------------- misc: fix building issue (undefined variables) Signed-off-by: Pierre-Yves Péneau <first.last@lirmm.fr> Reviewed-by: Andreas Hansson <andreas.hansson@arm.com> Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Posted (Feb. 3, 2017, 12:57 a.m.)
-
src/mem/comm_monitor.cc (Diff revision 1) -
the (const bool) part seems excessive
Posted (Feb. 3, 2017, 12:58 a.m.)
Thanks for fixing. Could you make it "mem:" and also include CommMonitor in the description. I also expect this is due to unused variables in fast builds? Not undefined variables.
Review request changed
Updated (Feb. 3, 2017, 1:02 a.m.)
Diff: |
Revision 2 (+4 -8) |
|---|
Review request changed
Updated (Feb. 3, 2017, 1:04 a.m.)
Summary: |
|
|---|
Ship It!
Ship It!
Review request changed
Updated (Feb. 4, 2017, 11:33 a.m.)
Description: |
|
|---|
