mem, misc: fix building issue with CommMonitor (unused variables)

Information
Submitter:	Pierre-Yves Péneau
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

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>

Testing Done

Issue Summary

Description	From	Last Updated	Status
the (const bool) part seems excessive	Andreas Hansson	Feb. 3, 2017, 1:06 a.m.	Resolved

src/mem/comm_monitor.cc (Diff revision 1)

the (const bool) part seems excessive

Show all issues

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.

Pierre-Yves Péneau Feb. 3, 2017, 1:10 a.m. (Feb. 3, 2017, 1:10 a.m.)
```
Yes, this is unused variables, thank you for pointing this.
```

Diff:

Revision 2 (+4 -8)

Show changes

src/mem/comm_monitor.cc

Summary:

-	misc: fix building issue (undefined variables)
+	mem, misc: fix building issue with CommMonitor (unused variables)

Ship It!

Ship It!

Description:

		Changeset 11808:e0730e1d7afa

		misc: fix building issue (undefined variables)

~		Signed-off-by: Pierre-Yves Péneau first.last@lirmm.fr
	~	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

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Issue Summary

Diff:

Summary:

Description:

Changeset 11808:e0730e1d7afa

Status: Closed (submitted)