> On Feb. 3, 2017, 9:58 a.m., Andreas Hansson wrote: > > 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.
Yes, this is unused variables, thank you for pointing this. - Pierre-Yves ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/#review9386 ----------------------------------------------------------- On Feb. 3, 2017, 10:04 a.m., Pierre-Yves Péneau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3801/ > ----------------------------------------------------------- > > (Updated Feb. 3, 2017, 10:04 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11808:e0730e1d7afa > --------------------------- > misc: fix building issue (undefined variables) > > Signed-off-by: Pierre-Yves Péneau <[email protected]> > > > Diffs > ----- > > src/mem/comm_monitor.cc 63325e5b0a9d > > Diff: http://reviews.gem5.org/r/3801/diff/ > > > Testing > ------- > > > Thanks, > > Pierre-Yves Péneau > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
