----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/777/#review2249 -----------------------------------------------------------
Gabe, can you mark this patch as submitted? Ali committed this change some time back. - Nilay Vaish On July 11, 2011, 5:02 a.m., Gabe Black wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/777/ > ----------------------------------------------------------- > > (Updated July 11, 2011, 5:02 a.m.) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Description > ------- > > O3: Don't assert that nothing is pending in the decoder at the start of a > cycle. > > The decoder clears out incoming instructions from fetch on a squash, but there > may be instructions on the wire incoming from fetch which don't get cleared. > Because the decoder is considered squashing, it doesn't actually process any > more instructions as they come in. The instructions that were on their way > would then sit in the "insts" list until the next cycle, tripping qn assert. > It's ok for them to be there in that case because the decoder will just ignore > them. Because "insts" is a queue and can't be scanned to verify that all > instructions left on it are squashed, this change simply gets rid of the > check. > > > Diffs > ----- > > src/cpu/o3/decode_impl.hh 9f3fedee88e2 > > Diff: http://reviews.gem5.org/r/777/diff/ > > > Testing > ------- > > > Thanks, > > Gabe Black > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
