- 29 Nov, 2016 4 commits
-
-
jgruber authored
If the prototype's map is not marked as fast, it is made fast after setup, breaking current RegExp fast path checks. BUG=v8:5339 Review-Url: https://codereview.chromium.org/2537483002 Cr-Commit-Position: refs/heads/master@{#41336}
-
eostroukhov authored
BUG=none Review-Url: https://codereview.chromium.org/2540463003 Cr-Commit-Position: refs/heads/master@{#41335}
-
kozyatinskiy authored
V8 internally uses conversions.h to convert number to string, we can use these methods too instead of slow std::stringstream with std::locale. BUG=chromium:661497,v8:5551 R=dgozman@chromium.org Review-Url: https://codereview.chromium.org/2534013002 Cr-Commit-Position: refs/heads/master@{#41334}
-
henrique.ferreiro authored
TypedArrays need specific checks before calling OrdinaryDefineOwnProperty. BUG=v8:5328 Review-Url: https://codereview.chromium.org/2431223005 Cr-Commit-Position: refs/heads/master@{#41333}
-
- 28 Nov, 2016 36 commits
-
-
ishell authored
[ic] Use validity cells to protect keyed element stores against object's prototype chain modifications. ... instead of clearing of all the KeyedStoreICs which didn't always work. BUG=chromium:662907, v8:5561 TBR=verwaest@chromium.org, bmeurer@chromium.org Review-Url: https://codereview.chromium.org/2534613002 Cr-Commit-Position: refs/heads/master@{#41332}
-
jshin authored
Due to a typo in runtime-i18n.js, 'ç'(U+00E7) was not uppercased while '÷'(U+00F7) was incorrectly uppercased to '×'(U+00D7). Add a comprehensive test for Latin-1 supplemental block (U+00A0 ~ U+00FF). (they're special-cased for speed-up and needs to have a test for the range.). TEST=intl/general/case-mapping BUG=v8:5681 Review-Url: https://codereview.chromium.org/2533033003 Cr-Commit-Position: refs/heads/master@{#41331}
-
ishell authored
BUG=chromium:666046 Review-Url: https://codereview.chromium.org/2539503002 Cr-Commit-Position: refs/heads/master@{#41330}
-
adamk authored
R=jochen@chromium.org Review-Url: https://codereview.chromium.org/2473153004 Cr-Commit-Position: refs/heads/master@{#41329}
-
hpayer authored
BUG=chromium:648568 Review-Url: https://codereview.chromium.org/2537533002 Cr-Commit-Position: refs/heads/master@{#41328}
-
ishell authored
BUG=chromium:666046 Review-Url: https://codereview.chromium.org/2539493002 Cr-Commit-Position: refs/heads/master@{#41327}
-
ishell authored
BUG= Review-Url: https://codereview.chromium.org/2537523002 Cr-Commit-Position: refs/heads/master@{#41326}
-
rodolph.perfetta authored
BUG= Review-Url: https://codereview.chromium.org/2537453003 Cr-Commit-Position: refs/heads/master@{#41325}
-
rmcilroy authored
Revert of [turbofan] Utilize String comparison feedback. (patchset #1 id:1 of https://codereview.chromium.org/2523463002/ ) Reason for revert: Seems to regress speedometer on Ignition and doesn't cause any improvements elsewhere. BUG=chromium:668651 Original issue's description: > [turbofan] Utilize String comparison feedback. > > Make use of the previously introduced String feedback for compare > operations in TurboFan. > > R=jarin@chromium.org > BUG=v8:5267,v8:5400 > > Committed: https://crrev.com/5d4253ecfb6ddcbbd7eb5654e728efa9559284a2 > Cr-Commit-Position: refs/heads/master@{#41163} TBR=jarin@chromium.org,bmeurer@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=v8:5267,v8:5400 Review-Url: https://codereview.chromium.org/2531183003 Cr-Commit-Position: refs/heads/master@{#41324}
-
mstarzinger authored
This removes the supporting function to perform a range-lookup in the exception handler table for unoptimized code. Such tables are by now guaranteed to be empty, the deoptimizer cannot encounter this case. R=jarin@chromium.org Committed: https://crrev.com/1f27ed9d7da78904e0418364c6394f913eabbe70 Review-Url: https://codereview.chromium.org/2529343003 Cr-Original-Commit-Position: refs/heads/master@{#41318} Cr-Commit-Position: refs/heads/master@{#41323}
-
zhengxing.li authored
The reason: The CL #41255 (https://codereview.chromium.org/2520363002 ) reimplemnt the Number.prototype.toString and the added number-tostring test cases failed at x87. Similar to many previos fixing CLs, i.e.: CL #37371 (https://codereview.chromium.org/2111493002 ), the root reason is: The Gcc compiler and it's dependent C++ libraris on linux platform use x87 in extended 80-bit double precision by default. So the reimplemented DoubleToRadixCString() will generate extended 80-bit double precision result which isn't the expected standard 64-bit double precision value. Although modifying DoubleToRadixCString() function to manually do the 80-bit <--> 64-bit conversion for each double/float computation step can fix this issue, but it wll drop the DoubleToRadixCString() function's performance of other architectures. This CL put the failed number-tostring test cases into number-tostring-big-integer.js and disables it for x87. BUG= Review-Url: https://codereview.chromium.org/2532073002 Cr-Commit-Position: refs/heads/master@{#41322}
-
tebbi authored
Removed a redundant check: If completion is not normal, then #iterator cannot be undefined. Review-Url: https://codereview.chromium.org/2533803002 Cr-Commit-Position: refs/heads/master@{#41321}
-
vogelheim authored
BUG=chromium:663410 Review-Url: https://codereview.chromium.org/2533463002 Cr-Commit-Position: refs/heads/master@{#41320}
-
mstarzinger authored
Revert of [deoptimizer] Remove dead Code::LookupRangeInHandlerTable. (patchset #2 id:20001 of https://codereview.chromium.org/2529343003/ ) Reason for revert: Seems to break TSAN builds. https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20TSAN/builds/12897/steps/Check/logs/stack-traces Original issue's description: > [deoptimizer] Remove dead Code::LookupRangeInHandlerTable. > > This removes the supporting function to perform a range-lookup in the > exception handler table for unoptimized code. Such tables are by now > guaranteed to be empty, the deoptimizer cannot encounter this case. > > R=jarin@chromium.org > > Committed: https://crrev.com/1f27ed9d7da78904e0418364c6394f913eabbe70 > Cr-Commit-Position: refs/heads/master@{#41318} TBR=jarin@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review-Url: https://codereview.chromium.org/2536673002 Cr-Commit-Position: refs/heads/master@{#41319}
-
mstarzinger authored
This removes the supporting function to perform a range-lookup in the exception handler table for unoptimized code. Such tables are by now guaranteed to be empty, the deoptimizer cannot encounter this case. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2529343003 Cr-Commit-Position: refs/heads/master@{#41318}
-
jgruber authored
BUG=v8:5339 NOTRY=true Review-Url: https://codereview.chromium.org/2532013003 Cr-Commit-Position: refs/heads/master@{#41317}
-
clemensh authored
Before, the encoded variant was stored in the compiled module, and the decoded one in the debug info (per instance). The decoded table was a FixedArray of ByteArrays. Now, also the decoded table is a flat ByteArray, and it encodes whether it is encoded or decoded. This saves memory and allows to store encoded and decoded variant in the same field. The table is automatically decoded on the first use. This CL also removes some unused and unimplemented methods from WasmDebugInfo (probably merge artifacts). That class is now pretty much empty, but we might still need it for breakpoint support. R=titzer@chromium.org, ahaas@chromium.org Review-Url: https://codereview.chromium.org/2522953002 Cr-Commit-Position: refs/heads/master@{#41316}
-
jkummerow authored
Introducing a TF_BUILTIN macro that wraps CodeStubAssembler usage into a convenient interface (using a subclass under the hood). No changes since previous attempt; this was only reverted because it blocked another revert. Original review: https://codereview.chromium.org/2517833005/ TBR=ishell@chromium.org Review-Url: https://codereview.chromium.org/2529373002 Cr-Commit-Position: refs/heads/master@{#41315}
-
jarin authored
BUG=chromium:668760 Review-Url: https://codereview.chromium.org/2530403002 Cr-Commit-Position: refs/heads/master@{#41314}
-
petermarshall authored
All super constructor calls go through the ignition + turbofan pipeline, so this is dead code. BUG=v8:5657 Review-Url: https://codereview.chromium.org/2525233003 Cr-Commit-Position: refs/heads/master@{#41313}
-
yangguo authored
BUG=v8:5510 R=jgruber@chromium.org Review-Url: https://codereview.chromium.org/2535733002 Cr-Commit-Position: refs/heads/master@{#41312}
-
yangguo authored
BUG=v8:5510 R=jgruber@chromium.org Review-Url: https://codereview.chromium.org/2536573002 Cr-Commit-Position: refs/heads/master@{#41311}
-
Jochen Eisinger authored
R=verwaest@chromium.org TBR=marja@chromium.org, verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/2531233002 . Cr-Commit-Position: refs/heads/master@{#41310}
-
jochen authored
They're supposed to be stable across several parse passes, so we'll also store them in the associated SharedFunctionInfos To achieve this, the PreParser and Parser need to generated the same number of FunctionLiterals. To achieve this, we teach the PreParser about desuggaring of class literals. For regular functions, the function IDs are assigned in the order they occur in the source. For arrow functions, however, we only know that it's an arrow function after parsing the parameter list, and so the ID assigned to the arrow function is larger than the IDs assigned to functions defined in the parameter list. This implies that we have to reset the function ID counter to before the parameter list when re-parsing an arrow function. To be able to do this, we store the number of function literals found in the parameter list of arrow functions as well. BUG=v8:5589 Review-Url: https://codereview.chromium.org/2481163002 Cr-Commit-Position: refs/heads/master@{#41309}
-
yangguo authored
R=jgruber@chromium.org BUG=v8:5510 Review-Url: https://codereview.chromium.org/2532893002 Cr-Commit-Position: refs/heads/master@{#41308}
-
marja authored
ThreadedList is more memory-efficient than ZoneList. This also enables us to use ThreadedList when making Preparser track parameters (upcoming work). BUG=v8:5501 Review-Url: https://codereview.chromium.org/2531593002 Cr-Commit-Position: refs/heads/master@{#41307}
-
jkummerow authored
Reducing visual clutter. No changes since previous attempt; this was only reverted because it blocked another revert. Original review: https://codereview.chromium.org/2519093002/ TBR=ishell@chromium.org Review-Url: https://codereview.chromium.org/2532063002 Cr-Commit-Position: refs/heads/master@{#41306}
-
cbruni authored
R=machenbach@chromium.org BUG=v8:5677 NOTRY=true Review-Url: https://codereview.chromium.org/2528363002 Cr-Commit-Position: refs/heads/master@{#41305}
-
machenbach authored
BUG=v8:5435 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_compile_dbg_ng;master.tryserver.chromium.android:android_clang_dbg_recipe Review-Url: https://codereview.chromium.org/2373783002 Cr-Commit-Position: refs/heads/master@{#41304}
-
rmcilroy authored
The BytecodeGraphBuilder assumes that deoptimization is always enabled, so always enable it when compiling from bytecode. BUG=chromium:668654 Review-Url: https://codereview.chromium.org/2531683004 Cr-Commit-Position: refs/heads/master@{#41303}
-
ahaas authored
Up until now assertThrows allows to check the type field of an exception, which is, however, a custom field introduced in a single regression test. With the change assertThrows allows to check the message field of an exception, which is set for standard V8 exceptions by default. I use the new assertThrows to refactor test/mjsunit/wasm/divrem-trap.js R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2525313003 Cr-Commit-Position: refs/heads/master@{#41302}
-
cbruni authored
A missing @@IsConcatSpreadable check caused the fast path inside the slow path to be incorrect and follow the default concat strategy when the arguments arrays contain only doubles. BUG=chromium:668414 Review-Url: https://codereview.chromium.org/2527173002 Cr-Commit-Position: refs/heads/master@{#41301}
-
jochen authored
R=yangguo@chromium.org,machenbach@chromium.org BUG= Review-Url: https://codereview.chromium.org/2529333002 Cr-Commit-Position: refs/heads/master@{#41300}
-
jgruber authored
AsyncTaskEvents are not exposed through the inspector interface. BUG=v8:5530 Review-Url: https://codereview.chromium.org/2532693002 Cr-Commit-Position: refs/heads/master@{#41299}
-
bmeurer authored
This code is no longer used by full-codegen since all functions which use new.target, rest parameters or the internal this function binding now grow through Ignition first, and never tier up to fullcodegen. BUG=v8:5657 R=rmcilroy@chromium.org Review-Url: https://codereview.chromium.org/2528293002 Cr-Commit-Position: refs/heads/master@{#41298}
-
machenbach authored
BUG=v8:5603 NOTRY=true Review-Url: https://codereview.chromium.org/2534813002 Cr-Commit-Position: refs/heads/master@{#41297}
-