----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/484/#review855 -----------------------------------------------------------
src/arch/arm/isa/insts/str.isa <http://reviews.m5sim.org/r/484/#comment1222> This should go up above like execBase, decConstBase, etc. as members of the class instead of instances. src/arch/arm/isa/insts/str.isa <http://reviews.m5sim.org/r/484/#comment1223> Same as above. - Gabe On 2011-02-11 16:48:07, Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/484/ > ----------------------------------------------------------- > > (Updated 2011-02-11 16:48:07) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > ARM: Mark store conditionals as such. > > > Diffs > ----- > > src/arch/arm/isa/insts/str.isa 6548721032fa > src/arch/arm/isa/insts/swap.isa 6548721032fa > > Diff: http://reviews.m5sim.org/r/484/diff > > > Testing > ------- > > > Thanks, > > Ali > >
_______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
