> On May 12, 2015, 1:52 p.m., Nilay Vaish wrote: > > It seems we can have a table that maps an operator to the expected type. > > The code for checking and raising error need not be duplicated.
The patch has been updated to use a map. - Tony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2791/#review6197 ----------------------------------------------------------- On July 20, 2015, 10:43 a.m., Tony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2791/ > ----------------------------------------------------------- > > (Updated July 20, 2015, 10:43 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10891:5d9dcfdd26b9 > --------------------------- > slicc: improve support for prefix operations > > This patch fixes the type handling when prefix operations are used. > Previously > prefix operators would assume a void return type, which made it impossible to > combine prefix operations with other expressions. This patch allows SLICC > programmers to use prefix operations more naturally. > > > Diffs > ----- > > src/mem/slicc/ast/OperatorExprAST.py > ebb3d0737aa72ec4fa24b6af9cf9a6b2a1109d18 > > Diff: http://reviews.gem5.org/r/2791/diff/ > > > Testing > ------- > > > Thanks, > > Tony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
