----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/656/#review1277 -----------------------------------------------------------
This looks fine to me, but I'm confused... don't you delete this code completely two patches from now? Why bother changing it if you're going to get rid of it? - Steve On 2011-04-25 03:04:12, Gabe Black wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/656/ > ----------------------------------------------------------- > > (Updated 2011-04-25 03:04:12) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > ISA parser: Allow defining operand types with a ctype directly. > > > Diffs > ----- > > src/arch/isa_parser.py de679a068dd8 > > Diff: http://reviews.m5sim.org/r/656/diff > > > Testing > ------- > > > Thanks, > > Gabe > > _______________________________________________ gem5-dev mailing list gem5-dev@m5sim.org http://m5sim.org/mailman/listinfo/gem5-dev