- 20 Oct, 2016 38 commits
-
-
bjaideep authored
R=joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Review-Url: https://chromiumcodereview.appspot.com/2439673005 Cr-Commit-Position: refs/heads/master@{#40485}
-
kozyatinskiy authored
BUG=chromium:656823 R=dgozman@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2436783003 Cr-Commit-Position: refs/heads/master@{#40484}
-
kozyatinskiy authored
* introduced DebugInterface::PrepareStep and DebugInterface::ClearStepping method. Inspector calls these methods only on pause and not interseted in calling this for not current break_id so we don't need to expose debug interface with break_id argument and can only check that current break_id is valid. BUG=chromium:652939,v8:5510 R=yangguo@chromium.org,dgozman@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Review-Url: https://chromiumcodereview.appspot.com/2423153002 Cr-Commit-Position: refs/heads/master@{#40483}
-
neis authored
R=adamk@chromium.org, marja@chromium.org BUG= Review-Url: https://chromiumcodereview.appspot.com/2436073002 Cr-Commit-Position: refs/heads/master@{#40482}
-
vogelheim authored
Revert of Speedup access to global_proxy.* attributes/accessors. (patchset #6 id:160001 of https://codereview.chromium.org/2403003002/ ) Reason for revert: Revert, because of crbug.com/656959. Original issue's description: > Speedup access to global_proxy.* attributes/accessors. > > Using a global proxy (e.g. 'window.f', 'w.f' or 'this.f') is considerably slower than evaluating just 'f'. This CL aims to perform the necessary checks at compile time and inline the accesses. > > This is a follow-on CL to crrev.com/2369933005: > - The initial upload is crrev.com/2369933005 + a rebase. > - The remaining issues are the fixes requested by the reviewers on that CL. > > BUG=chromium:634276, chromium:654716 > > Committed: https://crrev.com/8f43d748272536117008aa6a1b53ea52126261c1 > Committed: https://crrev.com/041314524952a3c1bc71bd3beafbbb37319f1d22 > Cr-Original-Commit-Position: refs/heads/master@{#40153} > Cr-Commit-Position: refs/heads/master@{#40365} TBR=jochen@chromium.org,verwaest@chromium.org NOTRY=true NOPRESUBMIT=true BUG=chromium:634276, chromium:654716 Review-Url: https://chromiumcodereview.appspot.com/2434233002 Cr-Commit-Position: refs/heads/master@{#40481}
-
alph authored
Encapsulate nodes and samples into cpuProfile object. BUG=chromium:406277 Review-Url: https://chromiumcodereview.appspot.com/2425093002 Cr-Commit-Position: refs/heads/master@{#40480}
-
adamk authored
Move hole check logic from full-codegen into scope analysis, and store the "needs hole check" bit on VariableProxy. This makes it easy to re-use in any backend: it will be trivial to extend the use of this logic in, e.g., full-codegen variable stores. While changing the signatures of the variable loading/storing methods in Ignition, I took the liberty of replacing the verb "Visit" with "Build", since these are not part of AST visiting. BUG=v8:5460 Review-Url: https://chromiumcodereview.appspot.com/2411873004 Cr-Commit-Position: refs/heads/master@{#40479}
-
ulan authored
Reland "[heap] Start sweeper tasks after evacuation. (patchset #2 id:20001 of https://chromiumcodereview.appspot.com/2428043002/ )" The performance regression in crbug.com/657776 was not caused by this CL. This reverts commit 4490a760. BUG= Review-Url: https://chromiumcodereview.appspot.com/2437873004 Cr-Commit-Position: refs/heads/master@{#40478}
-
jpp authored
This CL adds support for: https://discourse.wicg.io/t/allow-const-global-variables/684 It allows global const variables to be used as if they were numeric literals. For example: const f0 = fround(0); ... function foo() { var v = f0; // no type annotation. ... return f0; // no return type annotation. } BUG= https://bugs.chromium.org/p/v8/issues/detail?id=5540 Review-Url: https://chromiumcodereview.appspot.com/2435823002 Cr-Commit-Position: refs/heads/master@{#40477}
-
heimbuef authored
Added a size constraint to the configuration to limit the segment pool. This will likely fix the memory alerts from small android devices. BUG=chromium:655129 Review-Url: https://chromiumcodereview.appspot.com/2424393002 Cr-Commit-Position: refs/heads/master@{#40476}
-
bjaideep authored
Port 99023682 Original commit message: The scheduler expects a trimmed graph, so we have to trim the graph before scheduling. R=ahaas@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Review-Url: https://chromiumcodereview.appspot.com/2431093004 Cr-Commit-Position: refs/heads/master@{#40475}
-
ahaas authored
The wasm specification does not fully specify the binary representation of NaN: the sign bit can be non-deterministic. The wasm-code fuzzer found a test case where the wasm interpreter and the compiled code produce a different sign bit for a NaN, and as a consequence they produce different results. With this CL the interpreter tracks whether it executed an instruction which can produce a NaN, which are div and sqrt instructions. The fuzzer uses this information and compares the result of the interpreter with the result of the compiled code only if there was no instruction which could have produced a NaN. R=titzer@chromium.org TEST=cctest/test-run-wasm-interpreter/TestMayProduceNaN BUG=chromium:657481 Review-Url: https://chromiumcodereview.appspot.com/2438603003 Cr-Commit-Position: refs/heads/master@{#40474}
-
mtrofin authored
We don't need the Deserialize API taking in a SerializedData anymore. BUG= Review-Url: https://chromiumcodereview.appspot.com/2436543004 Cr-Commit-Position: refs/heads/master@{#40473}
-
mtrofin authored
When allocating for splinters, we were prematurely reverting to the hot range behavior, even when the range didn't actually have any positions requiring a register. This could cause unnecessary moves. BUG= Review-Url: https://chromiumcodereview.appspot.com/2436813003 Cr-Commit-Position: refs/heads/master@{#40472}
-
mstarzinger authored
Breaks layout tests on Blink builders. TBR=machenbach@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2428413004 Cr-Commit-Position: refs/heads/master@{#40471}
-
mstarzinger authored
R=mythria@chromium.org BUG=v8:5038 Review-Url: https://chromiumcodereview.appspot.com/2437023002 Cr-Commit-Position: refs/heads/master@{#40470}
-
ishell authored
BUG= Review-Url: https://chromiumcodereview.appspot.com/2433333002 Cr-Commit-Position: refs/heads/master@{#40469}
-
mythria authored
Add support to collect feedback about oddballs for Bitwise binary operations and Increment and decrement operations. For the case of Oddballs the code to convert them to numbers is inlined into the handlers instead of calling the NonNumberToNumber Stub. BUG=v8:4280, v8:5400 Review-Url: https://chromiumcodereview.appspot.com/2407103003 Cr-Commit-Position: refs/heads/master@{#40468}
-
ivica.bogosavljevic authored
SEB and SEH instructions are not available on MIPS32R1. This caused several failures on MIPS32R1 in mjsunit/wasm/* and mjsunit/asm test suites. This fix simulates these instruction in MacroAssembler for those architectures that do not support them. TEST=mjsunit/asm/sqlite3/sqlite-pointer-masking,mjsunit/wasm/embenchen/lua_binarytrees BUG= Review-Url: https://chromiumcodereview.appspot.com/2434973002 Cr-Commit-Position: refs/heads/master@{#40467}
-
ishell authored
This CL also introduces IsSetWord<T>(..) and IsSetWord32<T>(..) operations to ease checking if the bit field is set or not. BUG= Review-Url: https://chromiumcodereview.appspot.com/2436893003 Cr-Commit-Position: refs/heads/master@{#40466}
-
ulan authored
Revert of [heap] Start sweeper tasks after evacuation. (patchset #2 id:20001 of https://chromiumcodereview.appspot.com/2428043002/ ) Reason for revert: Performance regression on arm64: crbug.com/657776 Original issue's description: > [heap] Start sweeper tasks after evacuation. > > This allows us to use more tasks for parallel evacuation. > > BUG= TBR=mlippautz@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/2440693002 Cr-Commit-Position: refs/heads/master@{#40465}
-
hablich authored
Revert of [inspector] align console implementation with spec (patchset #2 id:20001 of https://chromiumcodereview.appspot.com/2428473003/ ) Reason for revert: Breaks roll: https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/164610 See https://chromiumcodereview.appspot.com/2434733002/ for more context Original issue's description: > [inspector] align console implementation with spec > > console should be non enumerable. > > BUG=chromium:656826 > R=dgozman@chromium.org TBR=dgozman@chromium.org,kozyatinskiy@chromium.org NOTRY=true BUG=chromium:656826 Review-Url: https://chromiumcodereview.appspot.com/2435013002 Cr-Commit-Position: refs/heads/master@{#40464}
-
titzer authored
R=ahaas@chromium.org BUG=chromium:654231 Review-Url: https://chromiumcodereview.appspot.com/2439613003 Cr-Commit-Position: refs/heads/master@{#40463}
-
mstarzinger authored
This enables Ignition unconditionally for all code that is destined for optimization with TurboFan. This ensures all optimization attempts will go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is dried out in practice. R=mvstanton@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2427953002 Cr-Commit-Position: refs/heads/master@{#40462}
-
mstarzinger authored
This removes validation code for state value consistency within frame states in the {BytecodeGraphBuilder} that is no longer used. R=jarin@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2432043006 Cr-Commit-Position: refs/heads/master@{#40461}
-
littledan authored
Review-Url: https://chromiumcodereview.appspot.com/2435783002 Cr-Commit-Position: refs/heads/master@{#40460}
-
jkummerow authored
Review-Url: https://chromiumcodereview.appspot.com/2434753003 Cr-Commit-Position: refs/heads/master@{#40459}
-
titzer authored
R=ahaas@chromium.org BUG=chromium:654231 Review-Url: https://chromiumcodereview.appspot.com/2433313002 Cr-Commit-Position: refs/heads/master@{#40458}
-
ulan authored
Currently it is possible to get into a cycle of mark-compact -> memory reducer -> mark-compact -> memory reducer ... where the memory reducer does not free memory. This patch ensures that the memory reducer restarts only if the committed memory increased by sufficient amount after the last run. BUG= Review-Url: https://chromiumcodereview.appspot.com/2433933005 Cr-Commit-Position: refs/heads/master@{#40457}
-
jochen authored
R=danno@chromium.org BUG= Review-Url: https://chromiumcodereview.appspot.com/2435953002 Cr-Commit-Position: refs/heads/master@{#40456}
-
machenbach authored
Revert of [inspector] migrate stepping related methods to debug-interface (patchset #3 id:40001 of https://chromiumcodereview.appspot.com/2423153002/ ) Reason for revert: https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/10808 https://github.com/v8/v8/wiki/Blink-layout-tests Original issue's description: > [inspector] migrate stepping related methods to debug-interface > > * introduced DebugInterface::PrepareStep and DebugInterface::ClearStepping method. > Inspector calls these methods only on pause and not interseted in calling this for not current break_id so we don't need to expose debug interface with break_id argument and can only check that current break_id is valid. > > BUG=chromium:652939,v8:5510 > R=yangguo@chromium.org,dgozman@chromium.org TBR=yangguo@chromium.org,dgozman@chromium.org,kozyatinskiy@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:652939,v8:5510 Review-Url: https://chromiumcodereview.appspot.com/2441583002 Cr-Commit-Position: refs/heads/master@{#40455}
-
zhengxing.li authored
port 99023682 (r40446) original commit message: The scheduler expects a trimmed graph, so we have to trim the graph before scheduling. BUG= Review-Url: https://chromiumcodereview.appspot.com/2431213003 Cr-Commit-Position: refs/heads/master@{#40454}
-
mstarzinger authored
This makes the creation of frame states "before" and "after" node sequences explicit in the {BytecodeGraphBuilder}. This removes some complexity and also allows us to ellide redundant {Checkpoint} nodes before operations that don't actually eager deoptimize. In this change such redundant {Checkpoint} nodes have been removed for arguments object and rest array creation bytecodes. The frame states used in such {Checkpoint} nodes were actually bogus because they would resume bytecode execution before the {new.target} value is assigned to its respective variable. R=jarin@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2437683003 Cr-Commit-Position: refs/heads/master@{#40453}
-
machenbach authored
Revert of [profiler] Update cpu profile tracing format. (patchset #2 id:20001 of https://chromiumcodereview.appspot.com/2425093002/ ) Reason for revert: Breaks https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/10805 See also: https://github.com/v8/v8/wiki/Blink-layout-tests Original issue's description: > [profiler] Update cpu profile tracing format. > > Encapsulate nodes and samples into cpuProfile object. > > BUG=chromium:406277 TBR=caseq@chromium.org,lpy@chromium.org,yangguo@chromium.org,alph@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:406277 Review-Url: https://chromiumcodereview.appspot.com/2432373004 Cr-Commit-Position: refs/heads/master@{#40452}
-
gdeepti authored
BUG=chromium:644670 R=ahaas@chromium.org, titzer@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2438593003 Cr-Commit-Position: refs/heads/master@{#40451}
-
kozyatinskiy authored
* introduced DebugInterface::PrepareStep and DebugInterface::ClearStepping method. Inspector calls these methods only on pause and not interseted in calling this for not current break_id so we don't need to expose debug interface with break_id argument and can only check that current break_id is valid. BUG=chromium:652939,v8:5510 R=yangguo@chromium.org,dgozman@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2423153002 Cr-Commit-Position: refs/heads/master@{#40450}
-
v8-autoroll authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/e54cf97..ee7e988 Rolling v8/third_party/catapult: https://chromium.googlesource.com/external/github.com/catapult-project/catapul/+log/ae987bf..fb397f8 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2429233005 Cr-Commit-Position: refs/heads/master@{#40449}
-
aseemgarg authored
BUG=v8:4124 TEST:test-run-wasm-simd-lowering R=titzer@chromium.org,bradnelson@chromium.org,gdeepti@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2294743003 Cr-Commit-Position: refs/heads/master@{#40448}
-
- 19 Oct, 2016 2 commits
-
-
alph authored
Encapsulate nodes and samples into cpuProfile object. BUG=chromium:406277 Review-Url: https://chromiumcodereview.appspot.com/2425093002 Cr-Commit-Position: refs/heads/master@{#40447}
-
ahaas authored
The scheduler expects a trimmed graph, so we have to trim the graph before scheduling. R=titzer@chromium.org, bmeurer@chromium.org TEST=cctest/test-run-wasm/RunWasmCompiled_GraphTrimming Review-Url: https://chromiumcodereview.appspot.com/2428443002 Cr-Commit-Position: refs/heads/master@{#40446}
-