----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/377/#review638 -----------------------------------------------------------
There's one minor glitch, but otherwise it looks fine. src/arch/x86/isa/microops/regop.isa <http://reviews.m5sim.org/r/377/#comment860> I think you meant to use imm_name here. - Gabe On 2011-01-06 15:51:09, Brad Beckmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/377/ > ----------------------------------------------------------- > > (Updated 2011-01-06 15:51:09) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > x86: set IsCondControl flag for the appropriate microops > > > Diffs > ----- > > src/arch/x86/isa/microops/regop.isa 9f9e10967912 > src/arch/x86/isa/microops/seqop.isa 9f9e10967912 > > Diff: http://reviews.m5sim.org/r/377/diff > > > Testing > ------- > > > Thanks, > > Brad > >
_______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
