> On 2011-06-21 04:49:06, Korey Sewell wrote: > > src/arch/mips/isa/decoder.isa, line 1559 > > <http://reviews.m5sim.org/r/748/diff/3/?file=13061#file13061line1559> > > > > same as above > > > > Why would nmsub add and nmadd subtract?
Actually, in the official MIPS ISA documents, the nmadd/nmsub means "multiply, add/subtract, then negtive". In the earlier version, the code of nmsub_s is "Fd.sf = -1 * ((Fs.sf * Ft.sf) - Fr.sf);", but the brackets have been changed now. I think someone have updated the code carelessly. - Deyuan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/748/#review1347 ----------------------------------------------------------- On 2011-06-21 02:16:53, Deyuan Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/748/ > ----------------------------------------------------------- > > (Updated 2011-06-21 02:16:53) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > Make the newly gem5 support mips branch likely instruction again. > Fix 4 files: > src/arch/mips/isa/formats/branch.isa > src/cpu/inorder/inorder_dyn_inst.cc > src/cpu/inorder/resources/branch_predictor.cc > src/cpu/inorder/resources/fetch_seq_unit.cc > > > Diffs > ----- > > src/arch/mips/isa/decoder.isa 00766f5b8177 > src/arch/mips/isa/formats/branch.isa 00766f5b8177 > src/cpu/inorder/inorder_dyn_inst.cc 00766f5b8177 > src/cpu/inorder/resources/branch_predictor.cc 00766f5b8177 > src/cpu/inorder/resources/fetch_seq_unit.cc 00766f5b8177 > > Diff: http://reviews.m5sim.org/r/748/diff > > > Testing > ------- > > Tested. > > > Thanks, > > Deyuan > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
