- 12 Sep, 2016 30 commits
-
-
bjaideep authored
Backtrace() and backtrace_symbols() API are not supported on AIX and there are no user-mode equivalent API for the same. For now, disabling the call to those API. Currently this is preventing V8 to build on AIX. This debug functionality was added in https://codereview.chromium.org/2248393002 R=rmcilroy@chromium.org BUG= LOG=N Review-Url: https://codereview.chromium.org/2324453002 Cr-Commit-Position: refs/heads/master@{#39356}
-
v8-autoroll authored
Rolling v8/base/trace_event/common to 26eac53e73b09625e9b8f783b479e54fcae85f58 Rolling v8/build to 9d601d07a9adaa2719d114290e79084d429a7dab Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to f9c1216417fe8e8f99d861a4d9cf360ae1d8645e Rolling v8/tools/clang to 4ac706a04b38c05a8012257dc657e069134902c7 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2331963002 Cr-Commit-Position: refs/heads/master@{#39355}
-
ishell authored
BUG=v8:5269 Review-Url: https://codereview.chromium.org/2322373002 Cr-Commit-Position: refs/heads/master@{#39354}
-
ulan authored
Reland "[heap] Fix a formatting bug in --trace-incremental-marking. (patchset #2 id:20001 of https://codereview.chromium.org/2324623004/ )" This reverts commit 6c2a217e. The performance regression in crbug.com/645411 was not caused by the original CL. TBR=mlippautz@chromium.org Review-Url: https://codereview.chromium.org/2329983002 Cr-Commit-Position: refs/heads/master@{#39353}
-
neis authored
This adds partial support of exports to the runtime system and to the interpreter. It introduces a new HeapObject JSModule that maps each of the module's export names to a Cell containing the exported value. Several aspects of this implementation are subject to change in follow-up CLs. BUG=v8:1569 Committed: https://crrev.com/241a0412eed919395a2e163b30b9b66071ce5c17 Review-Url: https://codereview.chromium.org/2302783002 Cr-Original-Commit-Position: refs/heads/master@{#39341} Cr-Commit-Position: refs/heads/master@{#39352}
-
ahaas authored
With this CL the wasm-code-fuzzer first decodes and interprets the test case generated by the fuzzer. It then compiles the test case, but only executes the compiled instance if the interpretation of the test case was successful. If the compiled instance is executed, then the result of the execution is compared with the result of the interpretation. Additionally this CL refactors the CompileAndRunWasmModule function in wasm-module.cc to resuse code in the call to the interpreter. R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2321443002 Cr-Commit-Position: refs/heads/master@{#39351}
-
ishell authored
BUG=v8:5269 Review-Url: https://codereview.chromium.org/2325013003 Cr-Commit-Position: refs/heads/master@{#39350}
-
bmeurer authored
We shall not mix AVX and SSE instructions as that can cause performance regressions in some areas, so make sure to emit vsqrtsd instead of sqrtsd when AVX is enabled. R=ahaas@chromium.org Review-Url: https://codereview.chromium.org/2335603002 Cr-Commit-Position: refs/heads/master@{#39349}
-
ulan authored
This patch changes incremental marking work scheduling from combination of idle/delayed tasks to ordinary short-running tasks and moves more marking work from V8.Execute to tasks by accounting how much bytes were marked in tasks. BUG=chromium:616434 LOG=NO Review-Url: https://codereview.chromium.org/2321553002 Cr-Commit-Position: refs/heads/master@{#39348}
-
petermarshall authored
CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_linux_noi18n_rel_ng BUG=v8:5364 Review-Url: https://codereview.chromium.org/2313393005 Cr-Commit-Position: refs/heads/master@{#39347}
-
rmcilroy authored
Rework Runtime::FunctionForName to take a c-string instead of a v8::String so that the parser can parse native syntax runtime calls without doing on-the-fly internalization. Also adds a c-string variant of IntrinsicIndexForName for the same reasons. BUG=v8:5215,chromium:634953 Review-Url: https://codereview.chromium.org/2324803002 Cr-Commit-Position: refs/heads/master@{#39346}
-
neis authored
Revert of [modules] Basic support of exports (patchset #10 id:180001 of https://codereview.chromium.org/2302783002/ ) Reason for revert: Failures related to deopt. Original issue's description: > [modules] Basic support of exports > > This adds partial support of exports to the runtime system and > to the interpreter. It introduces a new HeapObject JSModule that > maps each of the module's export names to a Cell containing the > exported value. > > Several aspects of this implementation are subject to change in > follow-up CLs. > > BUG=v8:1569 > > Committed: https://crrev.com/241a0412eed919395a2e163b30b9b66071ce5c17 > Cr-Commit-Position: refs/heads/master@{#39341} TBR=adamk@chromium.org,rmcilroy@chromium.org,ulan@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:1569 Review-Url: https://codereview.chromium.org/2328283002 Cr-Commit-Position: refs/heads/master@{#39345}
-
neis authored
R=rmcilroy@chromium.org BUG= Review-Url: https://codereview.chromium.org/2331913002 Cr-Commit-Position: refs/heads/master@{#39344}
-
Alexander.Gilday2 authored
Migrate ToNumber platform builtin to TurboFan. Also move NonNumberToNumber builtin implementation to helper function. BUG=v8:5049 Review-Url: https://codereview.chromium.org/2327703003 Cr-Commit-Position: refs/heads/master@{#39343}
-
petermarshall authored
CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_linux_noi18n_rel_ng BUG=v8:5364 Committed: https://crrev.com/7f84a6a2e7000bebba49354b4648346ff606ca34 Review-Url: https://codereview.chromium.org/2315343002 Cr-Original-Commit-Position: refs/heads/master@{#39331} Cr-Commit-Position: refs/heads/master@{#39342}
-
neis authored
This adds partial support of exports to the runtime system and to the interpreter. It introduces a new HeapObject JSModule that maps each of the module's export names to a Cell containing the exported value. Several aspects of this implementation are subject to change in follow-up CLs. BUG=v8:1569 Review-Url: https://codereview.chromium.org/2302783002 Cr-Commit-Position: refs/heads/master@{#39341}
-
mstarzinger authored
It is invalid for OSR deconstruction to leave a graph with a node representing the OSR normal entry (and no OSR loop entry). Subsequent lowering phases will not handle {OsrNormalEntry} operators and hence will lead to serious clogging further down the pipeline. R=bmeurer@chromium.org BUG=chromium:641893 Review-Url: https://codereview.chromium.org/2336543002 Cr-Commit-Position: refs/heads/master@{#39340}
-
ahaas authored
With this CL the AstDecoder produces an error if it encounters a grow_memory instruction in an asmjs module. Additionally asmjs instructions are not allowed anymore in wasm modules. BUG=chromium:644674 R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2324733002 Cr-Commit-Position: refs/heads/master@{#39339}
-
bmeurer authored
No longer disable TurboFan for functions with eval, but only disable Crankshaft in those cases. R=mstarzinger@chromium.org BUG=v8:5267 Review-Url: https://codereview.chromium.org/2334633002 Cr-Commit-Position: refs/heads/master@{#39338}
-
nikolaos authored
This patch moves the following parsing method to ParserBase: - ParseSwitchStatement It also removes ParseCaseClause and merges it with ParseSwitchStatement, mainly to avoid the complexity of introducing one more abstract typedef to be shared between parser implementations, but also because the merged ParseSwitchStatement is now only 59 lines. R=adamk@chromium.org, marja@chromium.org BUG= LOG=N Review-Url: https://codereview.chromium.org/2324843005 Cr-Commit-Position: refs/heads/master@{#39337}
-
bmeurer authored
R=mvstanton@chromium.org Review-Url: https://codereview.chromium.org/2335533002 Cr-Commit-Position: refs/heads/master@{#39336}
-
ishell authored
Compare the whole word on 64-bit architectures with the kHoleNanInt64 value since comparing only upper part produces slightly worse code on x64. Currently TurboFan is not able to fold (array + ((const1 + index*scale) + const)) into single addressing mode: (array + (const + index*scale)). BUG=chromium:645414 Review-Url: https://codereview.chromium.org/2322283002 Cr-Commit-Position: refs/heads/master@{#39335}
-
mstarzinger authored
R=mvstanton@chromium.org Review-Url: https://codereview.chromium.org/2326493002 Cr-Commit-Position: refs/heads/master@{#39334}
-
neis authored
Remove the "Entry"/"Entries" suffix as it doesn't add any information and ModuleInfoEntry would clash with another name being introduced soon. R=adamk@chromium.org BUG= Review-Url: https://codereview.chromium.org/2300373002 Cr-Commit-Position: refs/heads/master@{#39333}
-
petermarshall authored
Revert of [builtins] Move StringNormalize to a cpp builtin. (patchset #10 id:180001 of https://codereview.chromium.org/2315343002/ ) Reason for revert: Tests fail when i18n is switched off, trybots do not run this configuration Original issue's description: > [builtins] Move StringNormalize to a cpp builtin. > > BUG=v8:5364 > > Committed: https://crrev.com/7f84a6a2e7000bebba49354b4648346ff606ca34 > Cr-Commit-Position: refs/heads/master@{#39331} TBR=bmeurer@chromium.org,franzih@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5364 Review-Url: https://codereview.chromium.org/2335553002 Cr-Commit-Position: refs/heads/master@{#39332}
-
petermarshall authored
BUG=v8:5364 Review-Url: https://codereview.chromium.org/2315343002 Cr-Commit-Position: refs/heads/master@{#39331}
-
ahaas authored
The use of the address of a local variable causes problems with asan, see https://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_triggered/builds/7803 R=jochen@chromium.org Review-Url: https://codereview.chromium.org/2317253006 Cr-Commit-Position: refs/heads/master@{#39330}
-
gdeepti authored
- Using realloc is still unsafe as the allocator, using array_buffer_allocator - Fixing tests to avoid overlapping stores, adding more tests BUG=v8:5344 R=ahaas@chromium.org, mlippautz@chromium.org Review-Url: https://codereview.chromium.org/2319983002 Cr-Commit-Position: refs/heads/master@{#39329}
-
bmeurer authored
The logic to test whether we already reached --max_inlining_levels when inlining into some optimized function only checked specifically for FrameStateType::kJavaScriptFunction, and thereby didn't properly account for FrameStateType::kInterpretedFunction, which is what we see when we come in via the bytecode pipeline. Review-Url: https://codereview.chromium.org/2329923002 Cr-Commit-Position: refs/heads/master@{#39328}
-
weiliang.lin authored
Both legacy and AVX versions BUG= Review-Url: https://codereview.chromium.org/2328843003 Cr-Commit-Position: refs/heads/master@{#39327}
-
- 10 Sep, 2016 5 commits
-
-
nikolaos authored
This patch moves the following parsing methods to ParserBase: - ParseDoExpression - ParseDoWhileStatement - ParseWhileStatement - ParseThrowStatement R=adamk@chromium.org, marja@chromium.org BUG= LOG=N Review-Url: https://codereview.chromium.org/2321103002 Cr-Commit-Position: refs/heads/master@{#39326}
-
nikolaos authored
This patch moves the following parsing methods to ParserBase: - ParseExpressionOrLabelledStatement - ParseIfStatement - ParseContinueStatement - ParseBreakStatement - ParseReturnStatement - ParseWithStatement R=adamk@chromium.org, marja@chromium.org BUG= LOG=N Review-Url: https://codereview.chromium.org/2323763002 Cr-Commit-Position: refs/heads/master@{#39325}
-
lpy authored
BUG=v8:5089 Review-Url: https://codereview.chromium.org/2329053002 Cr-Commit-Position: refs/heads/master@{#39324}
-
alph authored
NOTRY=true Review-Url: https://codereview.chromium.org/2325993002 Cr-Commit-Position: refs/heads/master@{#39323}
-
kozyatinskiy authored
BUG=chromium:642496 R=dgozman@chromium.org Review-Url: https://codereview.chromium.org/2323173004 Cr-Commit-Position: refs/heads/master@{#39322}
-
- 09 Sep, 2016 5 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://codereview.chromium.org/2321973006 Cr-Commit-Position: refs/heads/master@{#39321}
-
littledan authored
R=adamk BUG=v8:5167 Review-Url: https://codereview.chromium.org/2326963003 Cr-Commit-Position: refs/heads/master@{#39320}
-
kelvinjin authored
Escape sequences may now be written to a trace file (previously, any string with a valid escapable character would fail a check). Also, string properties are now surrounded with quotes. BUG=v8:4561 Review-Url: https://codereview.chromium.org/2309943005 Cr-Commit-Position: refs/heads/master@{#39319}
-
eholk authored
This CL introduces a ProtectedLoad instruction with is needed for out of bounds trap handling. ProtectedLoad behaves like a regular load, but it takes a context and source position parameter as well. These are used by an out of line code fragment to generate code to throw a JS exception for an out of bounds memory reference in Wasm. These changes a cleaned up subset of https://codereview.chromium.org/2148743004/ The rest of this feature will follow in future CLs. This includes a table mapping memory instructions to landing pads as well as the actual signal handler. BUG= https://bugs.chromium.org/p/v8/issues/detail?id=5277 Review-Url: https://codereview.chromium.org/2301833004 Cr-Commit-Position: refs/heads/master@{#39318}
-
adamk authored
The array spread operator is now handled by desugaring in the parser. Review-Url: https://codereview.chromium.org/2324013002 Cr-Commit-Position: refs/heads/master@{#39317}
-