AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[builtins] Drop the special MathRandomRaw.

Browse Source 
This is the last user of %_DoubleLo and actually unnecessary, since we
can just use the normal Math.random() instead.

R=yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2109483004
Cr-Commit-Position: refs/heads/master@{#37359}
This commit is contained in:
bmeurer 2016-06-28 23:19:24 -07:00 committed by Commit bot
parent 0e1eaec71d
commit 50e1954703
2 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/js/collection.js
View File

@ -14,7 +14,7 @@ var GlobalMap = global.Map;
var GlobalObject = global.Object; var GlobalObject = global.Object;
var GlobalSet = global.Set; var GlobalSet = global.Set;
var hashCodeSymbol = utils.ImportNow("hash_code_symbol"); var hashCodeSymbol = utils.ImportNow("hash_code_symbol");
var IntRandom; var MathRandom;
var MakeTypeError; var MakeTypeError;
var MapIterator; var MapIterator;
var NumberIsNaN; var NumberIsNaN;
@ -23,7 +23,7 @@ var speciesSymbol = utils.ImportNow("species_symbol");
var toStringTagSymbol = utils.ImportNow("to_string_tag_symbol"); var toStringTagSymbol = utils.ImportNow("to_string_tag_symbol");
utils.Import(function(from) { utils.Import(function(from) {
IntRandom = from.IntRandom; MathRandom = from.MathRandom;
MakeTypeError = from.MakeTypeError; MakeTypeError = from.MakeTypeError;
MapIterator = from.MapIterator; MapIterator = from.MapIterator;
NumberIsNaN = from.NumberIsNaN; NumberIsNaN = from.NumberIsNaN;
@ -113,7 +113,7 @@ function GetExistingHash(key) {
function GetHash(key) { function GetHash(key) {
var hash = GetExistingHash(key); var hash = GetExistingHash(key);
if (IS_UNDEFINED(hash)) { if (IS_UNDEFINED(hash)) {
hash = IntRandom() | 0; hash = (MathRandom() * 0x40000000) | 0;
if (hash === 0) hash = 1; if (hash === 0) hash = 1;
SET_PRIVATE(key, hashCodeSymbol, hash); SET_PRIVATE(key, hashCodeSymbol, hash);
} }

10
src/js/math.js
View File

@ -47,14 +47,6 @@ function MathRandom() {
return randomNumbers[--nextRandomIndex]; return randomNumbers[--nextRandomIndex];
} }
function MathRandomRaw() {
if (nextRandomIndex <= kRandomNumberStart) {
randomNumbers = %GenerateRandomNumbers(randomNumbers);
nextRandomIndex = %_TypedArrayGetLength(randomNumbers);
}
return %_DoubleLo(randomNumbers[--nextRandomIndex]) & 0x3FFFFFFF;
}
// ES6 draft 09-27-13, section 20.2.2.28. // ES6 draft 09-27-13, section 20.2.2.28.
function MathSign(x) { function MathSign(x) {
x = +x; x = +x;
@ -142,7 +134,7 @@ utils.InstallFunctions(GlobalMath, DONT_ENUM, [
utils.Export(function(to) { utils.Export(function(to) {
to.MathAbs = MathAbs; to.MathAbs = MathAbs;
to.IntRandom = MathRandomRaw; to.MathRandom = MathRandom;
}); });
}) })