Re: [gem5-dev] Review Request: ISA parser: Stop supporting the old style operand types.

2011-05-29 Thread Gabe Black


> On 2011-05-28 22:09:01, Steve Reinhardt wrote:
> > See previous reviews... this looks fine if all the ISAs are converted to 
> > use it.  Once it's committed, the relevant documentation should also be 
> > updated (e.g., http://gem5.org/Code_parsing#Operand_type_qualifiers).

Will do.


- Gabe


---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/658/#review1279
---


On 2011-04-25 03:04:33, Gabe Black wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/658/
> ---
> 
> (Updated 2011-04-25 03:04:33)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> ---
> 
> ISA parser: Stop supporting the old style operand types.
> 
> 
> Diffs
> -
> 
>   src/arch/isa_parser.py de679a068dd8 
> 
> Diff: http://reviews.m5sim.org/r/658/diff
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Gabe
> 
>

___
gem5-dev mailing list
gem5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/gem5-dev


Re: [gem5-dev] Review Request: ISA parser: Stop supporting the old style operand types.

2011-05-28 Thread Steve Reinhardt

---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/658/#review1279
---


See previous reviews... this looks fine if all the ISAs are converted to use 
it.  Once it's committed, the relevant documentation should also be updated 
(e.g., http://gem5.org/Code_parsing#Operand_type_qualifiers).

- Steve


On 2011-04-25 03:04:33, Gabe Black wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/658/
> ---
> 
> (Updated 2011-04-25 03:04:33)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> ---
> 
> ISA parser: Stop supporting the old style operand types.
> 
> 
> Diffs
> -
> 
>   src/arch/isa_parser.py de679a068dd8 
> 
> Diff: http://reviews.m5sim.org/r/658/diff
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Gabe
> 
>

___
gem5-dev mailing list
gem5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/gem5-dev