- 25 Nov, 2015 40 commits
-
-
littledan authored
In this new version of test262, a number of known failing tests have been changed to match the ES2016 semantics, which V8 implements. R=adamk Review URL: https://codereview.chromium.org/1475793004 Cr-Commit-Position: refs/heads/master@{#32303}
-
mtrofin authored
validating that the hot path does not spill - somewhat simpler code. Cleared the scenario where a range is defined in a deferred block. The code before was slightly more complicated by not leveraging the property that these sort of ranges would be completely contained within deferred blocks. Moved "spills in deferred blocks" marking to a more appropriate location. One thing this CL achieves is correct support for scenarios where a range is spilled both on the deferred and then hot path, and the ranges concatenate. I owe better unit testing, which I will add in a subsequent CL. BUG= Review URL: https://codereview.chromium.org/1472803004 Cr-Commit-Position: refs/heads/master@{#32302}
-
ahaas authored
The Float32RoundTruncate operator rounds float32 numbers towards zero. The operator is currently implemented on x64, ia32, arm, and arm64. Additionally I added support for the float32 vrintz, vrintn, and vrinta instructions to the arm simulator. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1468303005 Cr-Commit-Position: refs/heads/master@{#32301}
-
jochen authored
Always go through Assembler::FlushICache so we automatically use the simulator when running with a simulator. BUG=v8:2487 R=epertoso@chromium.org LOG=n Review URL: https://codereview.chromium.org/1480623002 Cr-Commit-Position: refs/heads/master@{#32300}
-
machenbach authored
Revert of [debugger] flood function for stepping before calling it. (patchset #7 id:120001 of https://codereview.chromium.org/1463803002/ ) Reason for revert: [Sheriff] Breaks layout tests: https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/3074 Original issue's description: > [debugger] flood function for stepping before calling it. > > R=verwaest@chromium.org > > Committed: https://crrev.com/93eb633214e0f97bf70ae30d2a07b7fbbaa78266 > Cr-Commit-Position: refs/heads/master@{#32285} TBR=verwaest@chromium.org,mstarzinger@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1474943005 Cr-Commit-Position: refs/heads/master@{#32299}
-
fedor authored
Merge BinaryOperatorReducer into the MachineOperatorReducer class. It does not need `Revisit()` calls, because the newly inserted nodes are visited anyway, and there are no other methods that need AdvancedReducer there. BUG= R=titzer@chromium.org Review URL: https://codereview.chromium.org/1473073004 Cr-Commit-Position: refs/heads/master@{#32298}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1472393002 Cr-Commit-Position: refs/heads/master@{#32297}
-
fmeawad authored
Since the gn builders do not resolve the DEPS after patch, the patch cannot contain both the DEPS change as well the usage of the newly added DEPS. If the CL mentioned in the title does not land, this CL is to be remove/reverted as well. BUG=v8:4560 LOG=N Review URL: https://codereview.chromium.org/1469303004 Cr-Commit-Position: refs/heads/master@{#32296}
-
ishell authored
BUG=v8:4572, chromium:561481 LOG=Y Review URL: https://codereview.chromium.org/1465223007 Cr-Commit-Position: refs/heads/master@{#32295}
-
machenbach authored
Revert of Tenure descriptor arrays. (patchset #1 id:1 of https://codereview.chromium.org/1476913002/ ) Reason for revert: [Sheriff] Breaks gc stress: https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20gc%20stress/builds/544 Original issue's description: > Tenure descriptor arrays. > > BUG= > > Committed: https://crrev.com/38ec44f3bde8c16448e093b6595d0452e189023f > Cr-Commit-Position: refs/heads/master@{#32289} TBR=ishell@chromium.org,hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1477003003 Cr-Commit-Position: refs/heads/master@{#32294}
-
machenbach authored
Reland of [turbofan] Introduce proper JSCreateLiteralRegExp operator. (patchset #1 id:1 of https://codereview.chromium.org/1472423002/ ) Reason for revert: [Sheriff] Wrong revert. Original issue's description: > Revert of [turbofan] Introduce proper JSCreateLiteralRegExp operator. (patchset #2 id:20001 of https://codereview.chromium.org/1475973002/ ) > > Reason for revert: > Broke "V8 Linux64 GC Stress - custom snapshot". > > http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/2824 > > Original issue's description: > > [turbofan] Introduce proper JSCreateLiteralRegExp operator. > > > > This adds a new JavaScript level operator for regexp literal creation, > > similar to what we already have for array and object literals. This > > once gets lowered to a call to the FastCloneRegExpStub always. > > > > R=mstarzinger@chromium.org > > > > Committed: https://crrev.com/8659c5d1d287177369ce179a8d0b910192d840d9 > > Cr-Commit-Position: refs/heads/master@{#32288} > > TBR=mstarzinger@chromium.org,bmeurer@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > > Committed: https://crrev.com/168683d270d44f05f28acaf7d34c32d0250d2a4c > Cr-Commit-Position: refs/heads/master@{#32292} TBR=mstarzinger@chromium.org,bmeurer@chromium.org,ishell@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1478823002 Cr-Commit-Position: refs/heads/master@{#32293}
-
ishell authored
Revert of [turbofan] Introduce proper JSCreateLiteralRegExp operator. (patchset #2 id:20001 of https://codereview.chromium.org/1475973002/ ) Reason for revert: Broke "V8 Linux64 GC Stress - custom snapshot". http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/2824 Original issue's description: > [turbofan] Introduce proper JSCreateLiteralRegExp operator. > > This adds a new JavaScript level operator for regexp literal creation, > similar to what we already have for array and object literals. This > once gets lowered to a call to the FastCloneRegExpStub always. > > R=mstarzinger@chromium.org > > Committed: https://crrev.com/8659c5d1d287177369ce179a8d0b910192d840d9 > Cr-Commit-Position: refs/heads/master@{#32288} TBR=mstarzinger@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1472423002 Cr-Commit-Position: refs/heads/master@{#32292}
-
mlippautz authored
Iterating live objects using mark bits is memory bound and already pretty heavy on the cache as we always need to look at the mark bits, the objects (payload), and their maps. This CL changes the object start computation from a table lookup, which was yet another memory location, to a CPU-bound computation. BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1478623003 Cr-Commit-Position: refs/heads/master@{#32291}
-
thakis authored
Also add a note that it should go away over time. BUG=none LOG=n Review URL: https://codereview.chromium.org/1475033003 Cr-Commit-Position: refs/heads/master@{#32290}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1476913002 Cr-Commit-Position: refs/heads/master@{#32289}
-
bmeurer authored
This adds a new JavaScript level operator for regexp literal creation, similar to what we already have for array and object literals. This once gets lowered to a call to the FastCloneRegExpStub always. R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1475973002 Cr-Commit-Position: refs/heads/master@{#32288}
-
jochen authored
R=jkummerow@chromium.org BUG=v8:2487 LOG=n Review URL: https://codereview.chromium.org/1473683004 Cr-Commit-Position: refs/heads/master@{#32287}
-
thakis authored
It appears unused, everything uses static_assert directly. BUG=none LOG=n Review URL: https://codereview.chromium.org/1465383006 Cr-Commit-Position: refs/heads/master@{#32286}
-
yangguo authored
R=verwaest@chromium.org Review URL: https://codereview.chromium.org/1463803002 Cr-Commit-Position: refs/heads/master@{#32285}
-
hpayer authored
BUG=561449 LOG=n Review URL: https://codereview.chromium.org/1468313007 Cr-Commit-Position: refs/heads/master@{#32284}
-
jochen authored
This is a preparation for requiring an isolate to construct a CodePatcher BUG=2487 R=epertoso@chromium.org LOG=n Review URL: https://codereview.chromium.org/1480573002 Cr-Commit-Position: refs/heads/master@{#32283}
-
machenbach authored
BUG=v8:4358 LOG=n TBR=yangguo@chromium.org, hablich@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1475053002 Cr-Commit-Position: refs/heads/master@{#32282}
-
ishell authored
BUG=v8:4531 LOG=Y Review URL: https://codereview.chromium.org/1468933004 Cr-Commit-Position: refs/heads/master@{#32281}
-
mythria authored
Adds implementation and tests for compare operators to bytecode graph builder. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1476473005 Cr-Commit-Position: refs/heads/master@{#32280}
-
jochen authored
Moving Blink from hidden values to private symbols almost doubled the rate of API calls, as the private symbols API increases the call-depth scope while the hidden values one didn't. I manually checked that this fixes the regression on the long running gmail benchmark but doesn't add new memory reducer GCs on the infinite scrolling benchmark BUG=chromium:561325 R=ulan@chromium.org LOG=n Review URL: https://codereview.chromium.org/1472193003 Cr-Commit-Position: refs/heads/master@{#32279}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#32278}
-
jochen authored
We always want to have an Isolate, so just use an extra ctor arg BUG=2487 R=yangguo@chromium.org,mstarzinger@chromium.org LOG=n Review URL: https://codereview.chromium.org/1476763002 Cr-Commit-Position: refs/heads/master@{#32277}
-
machenbach authored
BUG=v8:4573 LOG=n TBR=mstarzinger@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1465193006 Cr-Commit-Position: refs/heads/master@{#32276}
-
Benedikt Meurer authored
TBR=machenbach@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1475843003 . Cr-Commit-Position: refs/heads/master@{#32275}
-
machenbach authored
Warn if unknown benchmark configurations are triggered. NOTRY=true Review URL: https://codereview.chromium.org/1479483003 Cr-Commit-Position: refs/heads/master@{#32274}
-
bmeurer authored
ES6 section 12.2.8.1 states that flags for regular expression literals must be checked during parsing and invalid flags are early errors. This change adapts the Scanner and (Pre)Parser to act according to the spec. This is also a prerequisite to unify the handling of literal creation (for Objects, Arrays, Regexps, and at some point Classes). R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1472323002 Cr-Commit-Position: refs/heads/master@{#32273}
-
ulan authored
BUG=chromium:554488 LOG=NO Review URL: https://codereview.chromium.org/1470773003 Cr-Commit-Position: refs/heads/master@{#32272}
-
machenbach authored
This loads all test suites and status files to catch subtle syntax errors. It also checks basic status file integrity and common mistakes. NOTRY=true Review URL: https://codereview.chromium.org/1475663002 Cr-Commit-Position: refs/heads/master@{#32271}
-
machenbach authored
Started failing after https://codereview.chromium.org/1469313002 NOTRY=true TBR=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1476603003 Cr-Commit-Position: refs/heads/master@{#32270}
-
titzer authored
Turns out we've been putting garbage into code->constant_pool_offset for quite some time. R=jkummerow@chromium.org BUG= Review URL: https://codereview.chromium.org/1478713002 Cr-Commit-Position: refs/heads/master@{#32269}
-
ishell authored
BUG=v8:4572 LOG=Y Review URL: https://codereview.chromium.org/1481493003 Cr-Commit-Position: refs/heads/master@{#32268}
-
machenbach authored
BUG= Review URL: https://codereview.chromium.org/1466303005 Cr-Commit-Position: refs/heads/master@{#32267}
-
jarin authored
Review URL: https://codereview.chromium.org/1479503002 Cr-Commit-Position: refs/heads/master@{#32266}
-
verwaest authored
BUG=v8:4518 LOG=n Review URL: https://codereview.chromium.org/1472293002 Cr-Commit-Position: refs/heads/master@{#32265}
-
mstarzinger authored
This passes the new.target value in a register instead of through a side-channel via the construct stub. The interpreter entry trampoline stores this value in a bytecode register so that it can be accessed directly by the interpreter. The size of the interpreter stack frame hence grows by one slot. R=oth@chromium.org BUG=v8:4544 LOG=n Review URL: https://codereview.chromium.org/1469313002 Cr-Commit-Position: refs/heads/master@{#32264}
-