> On 2011-09-06 11:40:51, Nathan Binkert wrote: > > Out of curiosity, are comments not being handled by PLY? If that's the > > case, why? Check out slicc/parser.py to see how to deal with comments.
We're talking about comments in the C snippets that the ISA parser handles via regexes, not the ISA DSL itself. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/841/#review1494 ----------------------------------------------------------- On 2011-09-05 22:34:45, Gabe Black wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/841/ > ----------------------------------------------------------- > > (Updated 2011-09-05 22:34:45) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > ISA parser: Match /* */ and // style comments. > > Comments should not be scanned for operands, and we should look for both /* */ > style and // style. The regular expression is still not quite right because it > doesn't handle comments in strings, but it's closer. > > > Diffs > ----- > > src/arch/isa_parser.py 1f95c9a0bb2f > > Diff: http://reviews.m5sim.org/r/841/diff > > > Testing > ------- > > > Thanks, > > Gabe > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
