- 06 Aug, 2015 3 commits
-
-
https://crrev.com/7e53749df0a10f475404e86ef0ca8df02bb79e7aulan authored
This fixes memory regression caused by not reducing new-space size. BUG=chromium:517468 LOG=NO Review URL: https://codereview.chromium.org/1273083002 Cr-Commit-Position: refs/heads/master@{#30049}
-
hpayer authored
When allocation rate is low and we are close to the new space limit, we should perform a scavenge during idle time. BUG=chromium:517395 LOG=n Review URL: https://codereview.chromium.org/1272573006 Cr-Commit-Position: refs/heads/master@{#30044}
-
ulan authored
Idle notification should force marking to make progress. BUG= Review URL: https://codereview.chromium.org/1275573002 Cr-Commit-Position: refs/heads/master@{#30041}
-
- 05 Aug, 2015 2 commits
-
-
hpayer authored
BUG=chromium:507211 LOG=n Review URL: https://codereview.chromium.org/1226773003 Cr-Commit-Position: refs/heads/master@{#30027}
-
mstarzinger authored
R=hpayer@chromium.org Review URL: https://codereview.chromium.org/1256283003 Cr-Commit-Position: refs/heads/master@{#30020}
-
- 04 Aug, 2015 10 commits
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1267313002 Cr-Commit-Position: refs/heads/master@{#30015}
-
mlippautz authored
Combines: * https://codereview.chromium.org/1273483002/ * https://codereview.chromium.org/1266793003/ This reverts commit 0215fb56. BUG= Review URL: https://codereview.chromium.org/1265273002 Cr-Commit-Position: refs/heads/master@{#30014}
-
mstarzinger authored
This introduces a CopyFixedArrayAndGrow method on Factory that takes the "grow amount" instead of the "new size" as an argument. The new interface is safer because it allows for mutations by the GC that potentially trim the source array. This also fixes a bug in SharedFunctionInfo::AddToOptimizedCodeMap where the aformentioned scenario led to unused entries within the optimized code map. Note that FixedArray::CopySize is hereby deprecated because it is considered unsafe and should no longer be used. R=hpayer@chromium.org TEST=mjsunit/regress/regress-crbug-513507 BUG=chromium:513507 LOG=n Review URL: https://codereview.chromium.org/1255173006 Cr-Commit-Position: refs/heads/master@{#30012}
-
adamk authored
Revert of GC: Refactor public incremental marking interface in heap (patchset #6 id:100001 of https://codereview.chromium.org/1273483002/ ) Reason for revert: Fails on the MSAN builder: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20MSAN/builds/3580/steps/Check/logs/RegExpInterruption Likely due to lack of initialization of IncrementalMarking::gc_callback_flags_. Original issue's description: > GC: Refactor incremental marking interface from heap > > BUG= > > Committed: https://crrev.com/c9fcaeb336919ce4b76fded8c8059457e9820250 > Cr-Commit-Position: refs/heads/master@{#30009} TBR=ulan@chromium.org,hpayer@chromium.org,mlippautz@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1271823006 Cr-Commit-Position: refs/heads/master@{#30011}
-
mlippautz authored
BUG= Review URL: https://codereview.chromium.org/1273483002 Cr-Commit-Position: refs/heads/master@{#30009}
-
ulan authored
BUG=chromium:516425 LOG=NO Review URL: https://codereview.chromium.org/1271973002 Cr-Commit-Position: refs/heads/master@{#30008}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1259613006 Cr-Commit-Position: refs/heads/master@{#30007}
-
ulan authored
BUG=chromium:515873 LOG=NO Review URL: https://codereview.chromium.org/1263373002 Cr-Commit-Position: refs/heads/master@{#29996}
-
ulan authored
BUG=chromium:515873 LOG=NO Review URL: https://codereview.chromium.org/1257833005 Cr-Commit-Position: refs/heads/master@{#29994}
-
hpayer authored
That makes going to the page safe. Addresses can be in arbitrary locations of an object, e.g. in a large object but not on the first 1M page. BUG= Review URL: https://codereview.chromium.org/1256203004 Cr-Commit-Position: refs/heads/master@{#29991}
-
- 03 Aug, 2015 11 commits
-
-
ofrobots authored
The recursion between AllocateRaw* and SlowAllocateRaw makes incremental stepping very complicated. This patch removes the recursion. Follow-on patches will improve accounting of allocations done by incremental mark. See: https://codereview.chromium.org/1252053003/#msg5 BUG= R=hpayer@chromium.org Review URL: https://codereview.chromium.org/1265443003 Cr-Commit-Position: refs/heads/master@{#29989}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1261373006 Cr-Commit-Position: refs/heads/master@{#29987}
-
mlippautz authored
BUG= Review URL: https://codereview.chromium.org/1270873002 Cr-Commit-Position: refs/heads/master@{#29986}
-
yangguo authored
There is only one use case for it: String.prototype.search converts a string argument into a RegExp. The cache is used to avoid repeating that conversion. However, this does not make the added complexity worthwhile. Review URL: https://codereview.chromium.org/1267493006 Cr-Commit-Position: refs/heads/master@{#29985}
-
mlippautz authored
High promotion mode limits new space size in the case where many objects (high rate) are promoted. The result is frequent scavenger runs with low pause times. The approach conflicts with pretenuring which requires a larger new space size to collect enough feedback to be usable. Ultimately the goal is to have pretenuring catch allocation sites that have high promotion rates. Revert "Bring back high promotion mode to shrink young generation size when scavenging latency is high." This reverts commit 37be1d5e. Revert "GC. Delay/avoid entering high promotion mode" This reverts commit a5221d07. BUG=chromium:499507,chromium:506910 LOG=n Review URL: https://codereview.chromium.org/1247723003 Cr-Commit-Position: refs/heads/master@{#29983}
-
hpayer authored
BUG=chromium:454297 LOG=n Review URL: https://codereview.chromium.org/1268663004 Cr-Commit-Position: refs/heads/master@{#29979}
-
hpayer authored
Reland of land concurrent sweeping of code space. (patchset #1 id:1 of https://codereview.chromium.org/1263343002/) Reason for revert: Bogus revert. Original issue's description: > Revert of Reland concurrent sweeping of code space. (patchset #6 id:100001 of https://codereview.chromium.org/1242333002/) > > Reason for revert: > Reverted because 507840 came back on recent Chromecrash. Should not have committed this Cl. > > Original issue's description: > > Reland concurrent sweeping of code space. > > > > BUG= > > > > Committed: https://crrev.com/8516dccf6a561020441773c93c564dd4aa6ee59e > > Cr-Commit-Position: refs/heads/master@{#29967} > > TBR=jochen@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG= > > Committed: https://crrev.com/5c6e7d00438cc82a5584e3178d7dadf36e4a34f8 > Cr-Commit-Position: refs/heads/master@{#29975} TBR=jochen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1271773002 Cr-Commit-Position: refs/heads/master@{#29976}
-
hpayer authored
Revert of Reland concurrent sweeping of code space. (patchset #6 id:100001 of https://codereview.chromium.org/1242333002/) Reason for revert: Reverted because 507840 came back on recent Chromecrash. Should not have committed this Cl. Original issue's description: > Reland concurrent sweeping of code space. > > BUG= > > Committed: https://crrev.com/8516dccf6a561020441773c93c564dd4aa6ee59e > Cr-Commit-Position: refs/heads/master@{#29967} TBR=jochen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1263343002 Cr-Commit-Position: refs/heads/master@{#29975}
-
bbudge authored
Adds Int32x4, Bool32x4, Int16x8, Bool16x8, Int8x16, Bool8x16. Adds Simd128Value base heap object class. Changes heap/factory construction pattern to use arrays. Adds replaceLane functions to facilitate testing. NOPRESUBMIT=true (presubmit checks erroneously interpret array declaration in macro definition as variable size array.) LOG=Y BUG=v8:4124 Review URL: https://codereview.chromium.org/1250733005 Cr-Commit-Position: refs/heads/master@{#29974}
-
ulan authored
Original CL: https://crrev.com/5aaceefd9a84327ce6afeaea16557449bf83ce57 BUG=chromium:515721 LOG=NO Review URL: https://codereview.chromium.org/1264063003 Cr-Commit-Position: refs/heads/master@{#29968}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1242333002 Cr-Commit-Position: refs/heads/master@{#29967}
-
- 31 Jul, 2015 5 commits
-
-
mvstanton authored
Since we need the notion of a dummy vector ic, we can use that to avoid a special case of the IC constructor. Also, consolidate the two dummy ICs into one. BUG= Review URL: https://codereview.chromium.org/1268783004 Cr-Commit-Position: refs/heads/master@{#29956}
-
yangguo authored
R=cbruni@chromium.org Review URL: https://codereview.chromium.org/1265923002 Cr-Commit-Position: refs/heads/master@{#29951}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1262363002 Cr-Commit-Position: refs/heads/master@{#29950}
-
ulan authored
This partially brings back the heuristic from v8 4.44. BUG= Review URL: https://codereview.chromium.org/1269743002 Cr-Commit-Position: refs/heads/master@{#29947}
-
mlippautz authored
BUG= Review URL: https://codereview.chromium.org/1269753002 Cr-Commit-Position: refs/heads/master@{#29945}
-
- 30 Jul, 2015 4 commits
-
-
chunyang.dai authored
When enabling the v8 profiler (Using the following command parameters: --js-flags=--prof) or vtune profiling in chromium. it will break. This failure is introduced by this CL: https://codereview.chromium.org/1218863002. The reason is that V8 will enable the JITted code logging if --prof is set for V8. And under this condition, the function Logger::LogCodeObjects() will be invoked and it will trigger a mark-compact GC when deserializing the snapshot. This GC will use MemoryReducer to post a delay task by invoking V8Platform::CallDelayedOnForegroundThread() function. But at this point V8 isolation is still under initialization and the PerIsolationData of this isolation has not been created. (isolation_holder.cc:39~40 line). This leads to V8Platform::CallDelayedOnForegroundThread() failure because of segment fault. According to my understanding, I proposed the following fix. If the heap deserialization has not be completed, it does not post the delay task for next GC. BUG= Review URL: https://codereview.chromium.org/1270493002 Cr-Commit-Position: refs/heads/master@{#29937}
-
hpayer authored
BUG=chromium:454297 LOG=n Review URL: https://codereview.chromium.org/1265943002 Cr-Commit-Position: refs/heads/master@{#29930}
-
rmcilroy authored
Adds interpreter entry and exit trampoline builtins. Also implements the Return bytecode handler and fixes a few bugs in InterpreterAssembler highlighted by running on other architectures. MIPS and MIPS64 port contributed by Paul Lind (paul.lind@imgtec.com) BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1245133002 Cr-Commit-Position: refs/heads/master@{#29929}
-
ulan authored
The idle time handler should never return DONE or DO_SCAVENGE for background tabs. Upon receiving DONE chrome will stop sending idle notifications. BUG=chromium:515174 LOG=NO Review URL: https://codereview.chromium.org/1269583002 Cr-Commit-Position: refs/heads/master@{#29926}
-
- 28 Jul, 2015 1 commit
-
-
jochen authored
Original issue's description: > Remove ExternalArray, derived types, and element kinds > > BUG=v8:3996 > R=jarin@chromium.org, mvstanton@chromium.org, bmeurer@chromium.org > LOG=y > > Committed: https://crrev.com/607ef7c6009a24ebf195b4cab7b0b436c5afd21c > Cr-Commit-Position: refs/heads/master@{#29872} BUG=v8:3996 R=bmeurer@chromium.org LOG=y Review URL: https://codereview.chromium.org/1262583002 Cr-Commit-Position: refs/heads/master@{#29893}
-
- 27 Jul, 2015 3 commits
-
-
machenbach authored
Revert of Remove ExternalArray, derived types, and element kinds (patchset #5 id:80001 of https://codereview.chromium.org/1254623002/) Reason for revert: [Sheriff] Breaks several layout tests, e.g.: http://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2032/builds/1067 Several output lines change from PASS to FAIL. If the changes are intended, please land a needsmanualrebaseline change in blink first. Original issue's description: > Remove ExternalArray, derived types, and element kinds > > BUG=v8:3996 > R=jarin@chromium.org, mvstanton@chromium.org, bmeurer@chromium.org > LOG=y > > Committed: https://crrev.com/607ef7c6009a24ebf195b4cab7b0b436c5afd21c > Cr-Commit-Position: refs/heads/master@{#29872} TBR=bmeurer@chromium.org,hpayer@chromium.org,jarin@chromium.org,mvstanton@chromium.org,jochen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3996 Review URL: https://codereview.chromium.org/1257223002 Cr-Commit-Position: refs/heads/master@{#29883}
-
jochen authored
BUG=v8:3996 R=jarin@chromium.org, mvstanton@chromium.org, bmeurer@chromium.org LOG=y Review URL: https://codereview.chromium.org/1254623002 Cr-Commit-Position: refs/heads/master@{#29872}
-
hpayer authored
BUG=chromium:507211 LOG=n Review URL: https://codereview.chromium.org/1260463002 Cr-Commit-Position: refs/heads/master@{#29864}
-
- 24 Jul, 2015 1 commit
-
-
rmcilroy authored
- Ensure frame_size is always set during allocation. - Add DCHECKs that frame_size is a valid value - Remove locals_count, which we don't need yet (possibly every) - Add a newline at the end of BytecodeArray::Dissassemble for each bytecode. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1254873002 Cr-Commit-Position: refs/heads/master@{#29852}
-