----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/157/#review172 -----------------------------------------------------------
Is this right? Seems like genrand(1) and genrand(2) will now do the same thing. Should we be calling ceilLog2(max+1) instead? - Steve On 2010-08-13 09:39:05, Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/157/ > ----------------------------------------------------------- > > (Updated 2010-08-13 09:39:05) > > > Review request for Default. > > > Summary > ------- > > BASE: Fix genrand to generate both 0s and 1s when max equals one. > previously was only generating 0s. > > > Diffs > ----- > > src/base/random.cc 3c48b2b3cb83 > > Diff: http://reviews.m5sim.org/r/157/diff > > > Testing > ------- > > > Thanks, > > Ali > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
