oooph...32/4 =8...8 bits in a byte...I plead the 5th! Originally, that code came from trying to align with some of the SPARC generic constants (SingleWidth) and I got a little too cute trying to define a SingleBytes.
And since no ALPHA or MIPS code depends on SingleBytes that's probably why that bug hasnt surfaced... On Sat, Feb 21, 2009 at 11:20 PM, nathan binkert <[email protected]> wrote: >> diff -r 7a74edaa8741 -r b6e4240c46e4 src/arch/alpha/floatregfile.hh >> --- a/src/arch/alpha/floatregfile.hh Sun Feb 15 23:43:39 2009 -0800 >> +++ b/src/arch/alpha/floatregfile.hh Fri Feb 20 09:06:11 2009 -0500 >> @@ -48,6 +48,13 @@ getFloatRegName(RegIndex) >> return ""; >> } >> >> +const int SingleWidth = 32; >> +const int SingleBytes = SingleWidth / 4; >> +const int DoubleWidth = 64; >> +const int DoubleBytes = DoubleWidth / 4; >> +const int QuadWidth = 128; >> +const int QuadBytes = QuadWidth / 4; >> + >> class FloatRegFile >> { >> public: > > I don't really know anything about this code, but if looks like you're > converting bits to bytes but dividing by 4 and not 8. Is this code > correct? > > Nate > _______________________________________________ > m5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/m5-dev > -- ---------- Korey L Sewell Graduate Student - PhD Candidate Computer Science & Engineering University of Michigan _______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
