- 04 Nov, 2015 15 commits
-
-
yangguo authored
Revert of Implement flag and source getters on RegExp.prototype. (patchset #3 id:50001 of https://codereview.chromium.org/1419823010/ ) Reason for revert: Performance issue. Original issue's description: > Implement flag and source getters on RegExp.prototype. > > R=littledan@chromium.org > BUG=v8:3715, v8:4528 > LOG=Y > > Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1 > Cr-Commit-Position: refs/heads/master@{#31753} > > Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5 > Cr-Commit-Position: refs/heads/master@{#31773} TBR=littledan@chromium.org,jochen@chromium.org,ulan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3715, v8:4528 Review URL: https://codereview.chromium.org/1409013006 Cr-Commit-Position: refs/heads/master@{#31780}
-
jarin authored
We don't really care about signedness when truncating to Word32. Review URL: https://codereview.chromium.org/1417023005 Cr-Commit-Position: refs/heads/master@{#31779}
-
mstarzinger authored
This makes the description in the README file for our test262 harness independent of the underlying revision. The one canonical place to mention the up-to-date revision is the DEPS file. R=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1424043006 Cr-Commit-Position: refs/heads/master@{#31778}
-
machenbach authored
BUG=chromium:535160 LOG=n Review URL: https://codereview.chromium.org/1413033014 Cr-Commit-Position: refs/heads/master@{#31777}
-
mstarzinger authored
This removes special casing for the 'f.arguments' property accessor. Any local 'arguments' variable should not be allowed to influence the value returned by the indirect 'f.arguments' property. That property creates a new object with a separate identity everytime it is read. This is by now consistent with other browsers. R=rossberg@chromium.org TEST=mjsunit/arguments-indirect Review URL: https://codereview.chromium.org/1408983006 Cr-Commit-Position: refs/heads/master@{#31776}
-
jkummerow authored
Review URL: https://codereview.chromium.org/1414403003 Cr-Commit-Position: refs/heads/master@{#31775}
-
rmcilroy authored
Adds support for calling JS runtime functions. Also changes the bytecode array builder to allow calling functions with an invalid argument register if the call takes no arguments. Adds the bytecode CallJSRuntime. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1410003003 Cr-Commit-Position: refs/heads/master@{#31774}
-
yangguo authored
R=littledan@chromium.org BUG=v8:3715, v8:4528 LOG=Y Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1 Cr-Commit-Position: refs/heads/master@{#31753} Review URL: https://codereview.chromium.org/1419823010 Cr-Commit-Position: refs/heads/master@{#31773}
-
bmeurer authored
Use the type specified for StoreField and StoreElement to check whether we are storing a TaggedSigned value, and skip the write barrier for the write completely in that case. Also make sure to set that field type appropriately for JSGlobalObjectSpecialization. R=jarin@chromium.org BUG=v8:4470 LOG=n Review URL: https://codereview.chromium.org/1410743009 Cr-Commit-Position: refs/heads/master@{#31772}
-
bmeurer authored
We don't need to distinguish between signed and unsigned integral representations in the big boy type system. It actually even hurts in some cases. The representation is only about the way the values are mapped to bits in memory/registers, but the interpretation of the bits is specified by the semantic dimension. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1416953006 Cr-Commit-Position: refs/heads/master@{#31771}
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1422623013 Cr-Commit-Position: refs/heads/master@{#31770}
-
mtrofin authored
We used to mark a block as needing frame based solely on the spill list. With splintering, that is not entirely accurate. With this change, for ranges that spill only in deferred blocks, we mark the start of each block in which a child range spills as needing a frame. BUG=v8:4533 LOG=n Review URL: https://codereview.chromium.org/1408183007 Cr-Commit-Position: refs/heads/master@{#31769}
-
v8-autoroll authored
Rolling v8/build/gyp to 2c1e6cced23554ce84806e570acea637f6473afc TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1413923012 Cr-Commit-Position: refs/heads/master@{#31768}
-
adamk authored
The "harmony"-prefixed files have been included in the snapshot for several releases now, and were only separate originally to enable loading them via a runtime flag. This patch simply merges them into the main implementation files for Arrays and TypedArrays, respectively. Review URL: https://codereview.chromium.org/1416243007 Cr-Commit-Position: refs/heads/master@{#31767}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1430943004 Cr-Commit-Position: refs/heads/master@{#31766}
-
- 03 Nov, 2015 25 commits
-
-
mlippautz authored
Revert of [heap] Turn on parallel compaction (patchset #1 id:1 of https://codereview.chromium.org/1364693002/ ) Reason for revert: Fails on gc stress https://chromegw.corp.google.com/i/client.v8/builders/V8%20Linux%20-%20gc%20stress/builds/157/ Original issue's description: > [heap] Turn on parallel compaction > > R=hpayer@chromium.org > BUG=chromium:524425 > LOG=N > > Committed: https://crrev.com/04db5bfa915766b228218ddc748af308b57ae8ea > Cr-Commit-Position: refs/heads/master@{#31763} TBR=hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:524425 Review URL: https://codereview.chromium.org/1424313008 Cr-Commit-Position: refs/heads/master@{#31765}
-
bradnelson authored
Only cast to integer with xor (closer to the spec which allows only ~~). Check type matching on the bitwise operations. Prevent mixing of types with the arthimetic operations. BUG= https://code.google.com/p/v8/issues/detail?id=4203 TEST=test-asm-validator R=titzer@chromium.org,aseemgarg@chromium.org LOG=N Review URL: https://codereview.chromium.org/1405383007 Cr-Commit-Position: refs/heads/master@{#31764}
-
mlippautz authored
R=hpayer@chromium.org BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1364693002 Cr-Commit-Position: refs/heads/master@{#31763}
-
mlippautz authored
BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1410633005 Cr-Commit-Position: refs/heads/master@{#31762}
-
balazs.kilvady authored
BUG= Review URL: https://codereview.chromium.org/1396133002 Cr-Commit-Position: refs/heads/master@{#31761}
-
ishell authored
BUG=v8:3886 LOG=Y Review URL: https://codereview.chromium.org/1422853004 Cr-Commit-Position: refs/heads/master@{#31760}
-
ishell authored
BUG=v8:3101, v8:3330 LOG=Y Review URL: https://codereview.chromium.org/1424283002 Cr-Commit-Position: refs/heads/master@{#31759}
-
machenbach authored
Revert of Implement flag and source getters on RegExp.prototype. (patchset #3 id:50001 of https://codereview.chromium.org/1419823010/ ) Reason for revert: [Sheriff] Changes layout tests. Please rebase upstream first. E.g.: http://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/2686 Original issue's description: > Implement flag and source getters on RegExp.prototype. > > R=littledan@chromium.org > BUG=v8:3715, v8:4528 > LOG=Y > > Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1 > Cr-Commit-Position: refs/heads/master@{#31753} TBR=littledan@chromium.org,jochen@chromium.org,ulan@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3715, v8:4528 Review URL: https://codereview.chromium.org/1427733005 Cr-Commit-Position: refs/heads/master@{#31758}
-
rmcilroy authored
Corrects LdaGlobal to deal with TypeofMode::INSIDE_TYPEOF so that it doesn't throw a reference error on undefined globals. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1422443006 Cr-Commit-Position: refs/heads/master@{#31757}
-
ishell authored
Original issue's description: > [es6] Fix Function and GeneratorFunction built-ins subclassing. > > BUG=v8:3101, v8:3330 > LOG=Y > > Committed: https://crrev.com/99e7f872d3d0a5fb799dcbafb05537cda491314a > Cr-Commit-Position: refs/heads/master@{#31708} The problem was in another CL, this is a clean reland with improved tests. BUG=v8:3101, v8:3330 LOG=Y Review URL: https://codereview.chromium.org/1415683007 Cr-Commit-Position: refs/heads/master@{#31756}
-
adamk authored
This matches the approach used for @@isConcatSpreadable, and seems to match what Mozilla is planning to do in Firefox. Given that there's already little compatibility around cross-origin toString results, there seems to be little hazard in making this change even before spec language hits the HTML spec. BUG=v8:3502, v8:4289, chromium:532469 LOG=n Review URL: https://codereview.chromium.org/1432543002 Cr-Commit-Position: refs/heads/master@{#31755}
-
ishell authored
Review URL: https://codereview.chromium.org/1432493003 Cr-Commit-Position: refs/heads/master@{#31754}
-
yangguo authored
R=littledan@chromium.org BUG=v8:3715, v8:4528 LOG=Y Review URL: https://codereview.chromium.org/1419823010 Cr-Commit-Position: refs/heads/master@{#31753}
-
ahaas authored
Review URL: https://codereview.chromium.org/1424333002 Cr-Commit-Position: refs/heads/master@{#31752}
-
neis authored
If the property is a data property on the holder (or does not exist) and is a readonly data property in the receiver, then we must fail. R=rossberg, verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/1424233005 Cr-Commit-Position: refs/heads/master@{#31751}
-
jkummerow authored
BUG=v8:4534 LOG=n R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1413723011 Cr-Commit-Position: refs/heads/master@{#31750}
-
mstarzinger authored
R=bmeurer@chromium.org TEST=mozilla/js1_5/Regress/regress-343713 Review URL: https://codereview.chromium.org/1424313007 Cr-Commit-Position: refs/heads/master@{#31749}
-
bmeurer authored
The JSNativeContextSpecialization class is getting rather huge with all the stuff related to property and element access going in. Splitting off the global object related stuff into JSGlobalObjectSpecialization seems like a natural separation, especially since the global object specialization is sort of separate issue anyway. This is neutral functionality- and performance-wise. R=jarin@chromium.org BUG=v8:4470 LOG=n Review URL: https://codereview.chromium.org/1417043006 Cr-Commit-Position: refs/heads/master@{#31748}
-
rossberg authored
R=neis@chromium.org BUG= Review URL: https://codereview.chromium.org/1422803003 Cr-Commit-Position: refs/heads/master@{#31747}
-
mstarzinger authored
R=yangguo@chromium.org TEST=cctest/test-debug/Backtrace Review URL: https://codereview.chromium.org/1415463017 Cr-Commit-Position: refs/heads/master@{#31746}
-
neis authored
R=verwaest@chromium.org BUG=chromium:548194 LOG=y Review URL: https://codereview.chromium.org/1426293003 Cr-Commit-Position: refs/heads/master@{#31745}
-
ishell authored
Review URL: https://codereview.chromium.org/1410023013 Cr-Commit-Position: refs/heads/master@{#31744}
-
ishell authored
1) The Map::CopyInitialMap() did not set descriptor's array if the source initial map had one. 2) Subclasses are temporarily disallowed to have more in-object properties than the parent class (for GC reasons). BUG=v8:3101, v8:3330, v8:4531 LOG=N Review URL: https://codereview.chromium.org/1431593003 Cr-Commit-Position: refs/heads/master@{#31743}
-
mstarzinger authored
This changes the inlining candidates to be stored in a sorted set of unique entries instead of a vector. We can avoid the final sorting operation by amortizing the cost across insertions and also duplicate entries are not created in the first place. Duplicate entries cause crashes when candidates are processed. R=bmeurer@chromium.org BUG=chromium:549113 LOG=n Review URL: https://codereview.chromium.org/1430553003 Cr-Commit-Position: refs/heads/master@{#31742}
-
rmcilroy authored
Existing code was assuming that 'lexical' blocks were the same as basic blocks, therefore code which emitted jumps within a lexical block (e.g., logical or) would in some occassions incorrectly omit a necessary ToBoolean. This change removes Enter/LeaveBlock from BytecodeArrayBuilder and instead tracks basic blocks via label bindings and jump operations. The change also ensures we don't emit dead code at the end of a basic block, and adds tests of the edge cases. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1406983010 Cr-Commit-Position: refs/heads/master@{#31741}
-