- 16 Nov, 2016 1 commit
-
-
mvstanton authored
In order to address a performance issue. BUG=v8:5512, chromium:664937 Review-Url: https://codereview.chromium.org/2506003002 Cr-Commit-Position: refs/heads/master@{#41041}
-
- 15 Nov, 2016 5 commits
-
-
Miran.Karic authored
When generating snapshot on a machine with a different page size than the target machine, we can run into problems as the v8 page area size changes. This is because v8 has page guards which depend on os page size, so if the target has larger os page, v8 page area is smaller and may not fit the contents. The solution proposed here is adding a flag, v8_os_page_size, that would, if used, override local os page size and use the one specified during snapshot generation. BUG= Review-Url: https://codereview.chromium.org/2488403003 Cr-Commit-Position: refs/heads/master@{#40997}
-
mlippautz authored
BUG=chromium:468240 NOTRY=true Review-Url: https://codereview.chromium.org/2498343002 Cr-Commit-Position: refs/heads/master@{#40991}
-
yangguo authored
R=peria@chromium.org, vogelheim@chromium.org BUG=chromium:617892 NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2495213003 Cr-Commit-Position: refs/heads/master@{#40981}
-
machenbach authored
Revert of [serializer] print use count of external references. (patchset #2 id:20001 of https://codereview.chromium.org/2495213003/ ) Reason for revert: Breaks https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20vtunejit/builds/14690 compile Original issue's description: > [serializer] print use count of external references. > > R=peria@chromium.org, vogelheim@chromium.org > BUG=chromium:617892 > NOPRESUBMIT=true TBR=peria@chromium.org,vogelheim@chromium.org,yangguo@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:617892 Review-Url: https://codereview.chromium.org/2498163003 Cr-Commit-Position: refs/heads/master@{#40980}
-
yangguo authored
R=peria@chromium.org, vogelheim@chromium.org BUG=chromium:617892 NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2495213003 Cr-Commit-Position: refs/heads/master@{#40979}
-
- 12 Nov, 2016 1 commit
-
-
jochen authored
That way we don't have to update the perf tests whenever the exact configuration changes. R=mvstanton@chromium.org,hablich@chromium.org,danno@chromium.org BUG= Review-Url: https://codereview.chromium.org/2420053002 Cr-Commit-Position: refs/heads/master@{#40944}
-
- 11 Nov, 2016 7 commits
-
-
jwolfe authored
BUG=v8:5051 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Review-Url: https://codereview.chromium.org/2484223004 Cr-Commit-Position: refs/heads/master@{#40942}
-
leszeks authored
Previous: https://codereview.chromium.org/2491773006 BUG=v8:5512 Review-Url: https://codereview.chromium.org/2498643002 Cr-Commit-Position: refs/heads/master@{#40937}
-
mlippautz authored
BUG=chromium:651354 Review-Url: https://codereview.chromium.org/2493803002 Cr-Commit-Position: refs/heads/master@{#40935}
-
ishell authored
BUG=v8:5561 Review-Url: https://codereview.chromium.org/2489433003 Cr-Commit-Position: refs/heads/master@{#40929}
-
rmcilroy authored
Revert of [compiler] Enable shared function marking by default (patchset #1 id:1 of https://codereview.chromium.org/2491773006/ ) Reason for revert: Revert because it depends on https://codereview.chromium.org/2478323002/ which has been reverted. Original issue's description: > [compiler] Enable shared function marking by default > > BUG=v8:5512 TBR=bmeurer@chromium.org,leszeks@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5512 Review-Url: https://codereview.chromium.org/2491643006 Cr-Commit-Position: refs/heads/master@{#40926}
-
leszeks authored
BUG=v8:5512 Review-Url: https://codereview.chromium.org/2491773006 Cr-Commit-Position: refs/heads/master@{#40920}
-
mstarzinger authored
This enables the on-stack replacement mechanism from Ignition bytecode to TurboFan optimized code by default. The runtime profiler is now allowed to arm OSR points in bytecode. R=rmcilroy@chromium.org Review-Url: https://codereview.chromium.org/2432413004 Cr-Commit-Position: refs/heads/master@{#40911}
-
- 10 Nov, 2016 1 commit
-
-
mlippautz authored
Also remove the flag scavenge_reclaim_unmodified_objects which has been defaulted to true for quite some time now. BUG=chromium:651354 Review-Url: https://codereview.chromium.org/2486173002 Cr-Commit-Position: refs/heads/master@{#40878}
-
- 09 Nov, 2016 1 commit
-
-
mstarzinger authored
This stages escape analysis by implying --turbo-escape by the --turbo flag. It broadens the exposure of the optimization in question. R=jarin@chromium.org BUG=v8:4586,v8:5267 Review-Url: https://codereview.chromium.org/1992913005 Cr-Commit-Position: refs/heads/master@{#40859}
-
- 08 Nov, 2016 2 commits
-
-
neis authored
With this CL, we set the is_source_positions_enabled flag on CompilationInfo when - a command line flag is enabled that requires Turbofan to preserve source position information (e.g. --trace-deopt), and - when profiling is enabled. This also removes the --turbo-source-positions flag. The goal is to eventually only track source position information when needed. R=mstarzinger@chromium.org BUG=v8:5439 Review-Url: https://codereview.chromium.org/2484163003 Cr-Commit-Position: refs/heads/master@{#40836}
-
lpy authored
This patch is a follow-up patch to enable gc statistics to use TracingCategoryObserver. Previously we need to pass --track_gc_object_stats to v8 if we want to enable gc statistics in tracing. In this patch, we introducce an integer flag FLAG_gc_stats, and FLAG_track_gc_object_stats and FLAG_trace_gc_object_stats will set it to 0x01, tracing will set it to 0x10 when we start tracing and reset the bit when we stop tracing. BUG=v8:5590 Review-Url: https://codereview.chromium.org/2459903003 Cr-Commit-Position: refs/heads/master@{#40817}
-
- 05 Nov, 2016 1 commit
-
-
machenbach authored
Revert of [Tracing] Use TracingCategoryObserver in gc statistics (patchset #5 id:100001 of https://codereview.chromium.org/2459903003/ ) Reason for revert: Speculative revert for blocking roll: https://codereview.chromium.org/2473003006/ E.g.: https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/331898 Original issue's description: > [Tracing] Use TracingCategoryObserver in gc statistics > > This patch is a follow-up patch to enable gc statistics to use > TracingCategoryObserver. > > Previously we need to pass --track_gc_object_stats to v8 if we want to enable > gc statistics in tracing. In this patch, we introducce an integer flag > FLAG_gc_stats, and FLAG_track_gc_object_stats and FLAG_trace_gc_object_stats > will set it to 0x01, tracing will set it to 0x10 when we start tracing and > reset the bit when we stop tracing. > > BUG=v8:5590 TBR=fmeawad@chromium.org,mlippautz@chromium.org,lpy@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5590 Review-Url: https://codereview.chromium.org/2477143002 Cr-Commit-Position: refs/heads/master@{#40785}
-
- 04 Nov, 2016 4 commits
-
-
adamk authored
The flag has been on since at least Chrome 53. R=littledan@chromium.org Review-Url: https://codereview.chromium.org/2478883002 Cr-Commit-Position: refs/heads/master@{#40780}
-
littledan authored
R=adamk@chromium.org BUG=v8:4806 Review-Url: https://codereview.chromium.org/2475383003 Cr-Commit-Position: refs/heads/master@{#40779}
-
lpy authored
This patch is a follow-up patch to enable gc statistics to use TracingCategoryObserver. Previously we need to pass --track_gc_object_stats to v8 if we want to enable gc statistics in tracing. In this patch, we introducce an integer flag FLAG_gc_stats, and FLAG_track_gc_object_stats and FLAG_trace_gc_object_stats will set it to 0x01, tracing will set it to 0x10 when we start tracing and reset the bit when we stop tracing. BUG=v8:5590 Review-Url: https://codereview.chromium.org/2459903003 Cr-Commit-Position: refs/heads/master@{#40777}
-
mstarzinger authored
This removes the deprecated flag in question which has been enabled by default a while ago. All components can by now deal with activations of a single function being mixed between Ignition and other compilers. The maintenance overhead to support a mode that clears bytecode is no longer warranted. R=rmcilroy@chromium.org BUG=v8:4280 Review-Url: https://codereview.chromium.org/2475203003 Cr-Commit-Position: refs/heads/master@{#40776}
-
- 02 Nov, 2016 1 commit
-
-
rmcilroy authored
BUG=v8:4280 Review-Url: https://codereview.chromium.org/2463353002 Cr-Commit-Position: refs/heads/master@{#40697}
-
- 31 Oct, 2016 2 commits
-
-
adamk authored
Both --harmony-object-values-entries and --harmony-object-own-property-descriptors are on by default in v8 5.4, which has now shipped to stable as Chrome 54. R=caitp@igalia.com Review-Url: https://codereview.chromium.org/2464733003 Cr-Commit-Position: refs/heads/master@{#40683}
-
bmeurer authored
R=mstarzinger@chromium.org BUG=v8:5267 Review-Url: https://codereview.chromium.org/2460333002 Cr-Commit-Position: refs/heads/master@{#40665}
-
- 28 Oct, 2016 3 commits
-
-
lpy authored
This patch implements TracingCategoryObserver to set global flag when a V8 specific category is enabled. Previously, we set a global flag each time when we encounter a top level trace event, and use it as a global check. With this patch, we can set a group of flags when tracing is enabled; besides, we make V8 tracing feature use V8 flags instead of defining its own flag in a messy way. With this patch, whatever V8 flag we want to imply in tracing, we define another integer flag, and the original V8 flag will set it to 0x01 when passing by commandline, tracing will set it to 0x10 when we start tracing and reset the bit when we stop tracing. Review-Url: https://codereview.chromium.org/2436273002 Cr-Commit-Position: refs/heads/master@{#40659}
-
ulan authored
The order is important for DEFINE_IMPLICATION to ensure transitive implication: predictable => single_threaded => !concurrent_sweeping. BUG=5580 Review-Url: https://codereview.chromium.org/2456323002 Cr-Commit-Position: refs/heads/master@{#40644}
-
mstarzinger authored
This flag is on by default for now. Whenever heuristics in the compiler pipeline decide to use Ignition+TurboFan, then {BytecodeGraphBuilder} is active. Removing the flag reduces maintenance overhead. R=mvstanton@chromium.org Review-Url: https://codereview.chromium.org/2437103002 Cr-Commit-Position: refs/heads/master@{#40639}
-
- 27 Oct, 2016 1 commit
-
-
leszeks authored
Reuses (and renames) the SFI "mark for optimization" flag to also permit marking for baseline recompilation. The flag now represents a "tier up" request, and CompileLazy can get baseline code as well as optimized code. BUG=v8:5512 Review-Url: https://codereview.chromium.org/2448933002 Cr-Commit-Position: refs/heads/master@{#40612}
-
- 26 Oct, 2016 1 commit
-
-
aseemgarg authored
R=titzer@chromium.org,binji@chromium.org BUG=v8:4614 Review-Url: https://codereview.chromium.org/2423883003 Cr-Commit-Position: refs/heads/master@{#40606}
-
- 25 Oct, 2016 1 commit
-
-
leszeks authored
This lets us investigate regressions caused by this marking while letting others continue their work without being impacted. BUG=v8:5512 Review-Url: https://codereview.chromium.org/2446673002 Cr-Commit-Position: refs/heads/master@{#40563}
-
- 24 Oct, 2016 1 commit
-
-
ishell authored
BUG= Review-Url: https://codereview.chromium.org/2438553003 Review-Url: https://codereview.chromium.org/2438553003 Cr-Original-Original-Commit-Position: refs/heads/master@{#40503} Cr-Original-Commit-Position: refs/heads/master@{#40511} Cr-Commit-Position: refs/heads/master@{#40524}
-
- 21 Oct, 2016 5 commits
-
-
ishell authored
Revert of [ic] Support data handlers that represent simple field stores. (patchset #2 id:40001 of https://codereview.chromium.org/2438553003/ ) Reason for revert: http://build.chromium.org/p/client.v8/builders/V8%20Mac%20GC%20Stress/builds/9306 "V8 Mac GC Stress" Original issue's description: > [ic] Support data handlers that represent simple field stores. > > BUG= > > Review-Url: https://codereview.chromium.org/2438553003 > Cr-Commit-Position: refs/heads/master@{#40503} TBR=jkummerow@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review-Url: https://chromiumcodereview.appspot.com/2442523003 Cr-Commit-Position: refs/heads/master@{#40512}
-
ishell authored
BUG= Review-Url: https://codereview.chromium.org/2438553003 Review-Url: https://chromiumcodereview.appspot.com/2438553003 Cr-Original-Commit-Position: refs/heads/master@{#40503} Cr-Commit-Position: refs/heads/master@{#40511}
-
jgruber authored
Revert of [ic] Support data handlers that represent simple field stores. (patchset #1 id:20001 of https://chromiumcodereview.appspot.com/2438553003/ ) Reason for revert: http://build.chromium.org/p/client.v8/builders/V8%20Mac%20GC%20Stress/builds/9299 Original issue's description: > [ic] Support data handlers that represent simple field stores. > > BUG= TBR=jkummerow@chromium.org,ishell@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review-Url: https://chromiumcodereview.appspot.com/2439053002 Cr-Commit-Position: refs/heads/master@{#40505}
-
ishell authored
BUG= Review-Url: https://chromiumcodereview.appspot.com/2438553003 Cr-Commit-Position: refs/heads/master@{#40503}
-
mlippautz authored
BUG= Review-Url: https://chromiumcodereview.appspot.com/2420063002 Cr-Commit-Position: refs/heads/master@{#40500}
-
- 18 Oct, 2016 1 commit
-
-
verwaest authored
BUG=v8:5501 Review-Url: https://codereview.chromium.org/2424013002 Cr-Commit-Position: refs/heads/master@{#40383}
-
- 17 Oct, 2016 1 commit
-
-
jkummerow authored
RIP, handwritten KeyedLoadICStub, handwritten KeyedLoadIC_Megamorphic, and hydrogenized KeyedLoadGeneric! Review-Url: https://codereview.chromium.org/2424433002 Cr-Commit-Position: refs/heads/master@{#40354}
-