----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/961/#review1797 -----------------------------------------------------------
src/cpu/o3/commit.hh <http://reviews.m5sim.org/r/961/#comment2332> I dont think instruction_in_flight is the right name here. You should update this to "macroop_in_flight" (or something of that nature) as well as update the comment. - Korey On 2011-12-30 16:38:31, Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/961/ > ----------------------------------------------------------- > > (Updated 2011-12-30 16:38:31) > > > Review request for Default. > > > Summary > ------- > > O3 CPU: Add a boolean flag for instruction midway in execution > This patch adds a boolean flag to the commit stage of the O3 cpu for > tracking whether or not there is an instruction that has committed some but > not all of its microops. This is required so that the interrupts are not > handled while an instruction is midway through its execution. > > > Diffs > ----- > > src/cpu/o3/commit.hh ca98021c3f96 > src/cpu/o3/commit_impl.hh ca98021c3f96 > src/cpu/o3/iew.hh ca98021c3f96 > > Diff: http://reviews.m5sim.org/r/961/diff > > > Testing > ------- > > > Thanks, > > Nilay > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
