> On Aug. 22, 2013, 10:42 p.m., Andreas Hansson wrote: > > src/cpu/o3/regfile.hh, line 114 > > <http://reviews.gem5.org/r/1983/diff/1/?file=37203#file37203line114> > > > > 0 <=? Can it be negative?
It's typedef'd to 'short int', so technically yes, but don't ask me why... > On Aug. 22, 2013, 10:42 p.m., Andreas Hansson wrote: > > src/cpu/o3/regfile.hh, line 240 > > <http://reviews.gem5.org/r/1983/diff/1/?file=37203#file37203line240> > > > > Is there a good reason these are not std::vectors? No need to > > change...just asking. No good reason, just history... but similarly no great benefit to changing at this point either, IMO. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1983/#review4622 ----------------------------------------------------------- On Aug. 22, 2013, 12:38 a.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1983/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2013, 12:38 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9846:816e0d9e97a0 > --------------------------- > cpu/o3: clean up physical register file > > No need for PhysRegFile to be a template class, or > have a pointer back to the CPU. Also made some methods > for checking the physical register type (int vs. float) > based on the phys reg index, which will come in handy later. > > > Diffs > ----- > > src/cpu/o3/cpu.cc 1ddfb0679c7569fb56382ac2187d6de038fd6f28 > src/cpu/o3/cpu_policy.hh 1ddfb0679c7569fb56382ac2187d6de038fd6f28 > src/cpu/o3/regfile.hh 1ddfb0679c7569fb56382ac2187d6de038fd6f28 > > Diff: http://reviews.gem5.org/r/1983/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
