For anybody that's familiar with ISA definitions, these changes refer specifically to the def operand_types dict, not the types maintained by the parser for specifying operands in def operands.
Gabe On 04/25/11 06:03, Gabe Black wrote: > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/655/ > ----------------------------------------------------------- > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > ISA parser: Simplify operand type handling. > > This change simplifies the code surrounding operand type handling and makes it > depend only on the ctype that goes with each operand type. Future changes will > allow defining operand types by their ctypes directly, convert the ISAs over > to that style of definition, and then remove support for the old style. These > changes are to make it easier to use non-builtin types like classes or > structures as the type for operands. > > > Diffs > ----- > > src/arch/alpha/isa/mem.isa de679a068dd8 > src/arch/arm/isa/insts/ldr.isa de679a068dd8 > src/arch/arm/isa/insts/mem.isa de679a068dd8 > src/arch/arm/isa/insts/str.isa de679a068dd8 > src/arch/arm/isa/templates/mem.isa de679a068dd8 > src/arch/isa_parser.py de679a068dd8 > src/arch/mips/isa/decoder.isa de679a068dd8 > src/arch/mips/isa/formats/mem.isa de679a068dd8 > src/arch/power/isa/decoder.isa de679a068dd8 > src/arch/power/isa/formats/mem.isa de679a068dd8 > src/arch/sparc/isa/decoder.isa de679a068dd8 > src/arch/sparc/isa/formats/mem/swap.isa de679a068dd8 > src/arch/sparc/isa/formats/mem/util.isa de679a068dd8 > > Diff: http://reviews.m5sim.org/r/655/diff > > > Testing > ------- > > > Thanks, > > Gabe > > _______________________________________________ > m5-dev mailing list > m5-dev@m5sim.org > http://m5sim.org/mailman/listinfo/m5-dev _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev