----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3805/#review9401 -----------------------------------------------------------
Seems to be the same thing as http://reviews.gem5.org/r/3801/. I guess Pierre-Yves beat you so we can drop this one? - Jason Lowe-Power On Feb. 3, 2017, 4:49 p.m., Curtis Dunham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3805/ > ----------------------------------------------------------- > > (Updated Feb. 3, 2017, 4:49 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > mem: fix fast build breakage from CommMonitor refactoring > > The is_read and is_write variables become unused in a fast > build (no DPRINTFs), leading to a -Werror=unused-variable > build failure. This fix simply eliminates the variables. > > Change-Id: I3f0f8596273f6d6c123087387440ea115eccbc16 > Reviewed-by: Andreas Sandberg <andreas.sandb...@arm.com> > > > Diffs > ----- > > src/mem/comm_monitor.cc 63325e5b0a9d > > Diff: http://reviews.gem5.org/r/3805/diff/ > > > Testing > ------- > > > Thanks, > > Curtis Dunham > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev