- 21 Mar, 2016 40 commits
-
-
ulan authored
STL requires comparison to be a strict weak ordering. In particular the predicate should be antisymmetric: f(x, y) implies !f(y, x). BUG=v8:4848 LOG=NO Review URL: https://codereview.chromium.org/1818043004 Cr-Commit-Position: refs/heads/master@{#34960}
-
mstarzinger authored
Now implemented as a builtin that delegates to the InstanceOfStub. That stub was parameterized to fallback to either Runtime_InstanceOf or to Runtime_OrdinaryHasInstance depending on the --harmony-instanceof flag. Once the feature stabilizes and the flag is no longer needed, we can get rid of this parameterization again. R=bmeurer@chromium.org BUG=v8:4447 LOG=n Review URL: https://codereview.chromium.org/1819813002 Cr-Commit-Position: refs/heads/master@{#34959}
-
vogelheim authored
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001 of https://codereview.chromium.org/1801203002/ ) Reason for revert: Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will revert for now. Original issue's description: > Parser: Make skipping HTML comments optional. > > API change: This adds a new flag skip_html_comments to v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour HTML-style comments in JS sources. > > (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular technique in the early days of JavaScript, to prevent non-JS-enabled browsers from displaying script sources to uses.) > > The flag defaults to 'true' when using v8::ScriptOrigin constructor, which preserves the existing behaviour. Embedders which are happy with the existing behaviour will thus not need any changes. > > BUG=chromium:573887 > LOG=Y > > Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173 > Cr-Commit-Position: refs/heads/master@{#34904} TBR=jochen@chromium.org,rossberg@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:573887, v8:4850 LOG=Y Review URL: https://codereview.chromium.org/1817163003 Cr-Commit-Position: refs/heads/master@{#34958}
-
ishell authored
We don't want them to disappear from the stack traces. BUG=v8:4698 LOG=N Review URL: https://codereview.chromium.org/1818063002 Cr-Commit-Position: refs/heads/master@{#34957}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1823613002 Cr-Commit-Position: refs/heads/master@{#34956}
-
oth authored
This change introduces wide prefix bytecodes to support wide (16-bit) and extra-wide (32-bit) operands. It retires the previous wide-bytecodes and reduces the number of operand types. Operands are now either scalable or fixed size. Scalable operands increase in width when a bytecode is prefixed with wide or extra-wide. The bytecode handler table is extended to 256*3 entries. The first 256 entries are used for bytecodes with 8-bit operands, the second 256 entries are used for bytecodes with operands that scale to 16-bits, and the third group of 256 entries are used for bytecodes with operands that scale to 32-bits. LOG=N BUG=v8:4747,v8:4280 Review URL: https://codereview.chromium.org/1783483002 Cr-Commit-Position: refs/heads/master@{#34955}
-
mstarzinger authored
The check in question will trigger because one of our unit tests still emits the bytecode (i.e. unittests/BytecodeArrayBuilderTest) even when the flag is turned on by default. R=rmcilroy@chromium.org TEST=unittests/BytecodeArrayBuilderTest.AllBytecodesGenerated Review URL: https://codereview.chromium.org/1820753003 Cr-Commit-Position: refs/heads/master@{#34954}
-
jyan authored
Generated code performs distinct floating multiply and add/subtract operations. Tests fail when GCC uses mul-add to calculate the expected result since these instructions provide higher accuracy due to the lack of an intermediate round. R=machenbach@chromium.org, mbrandy@us.ibm.com, joransiu@ca.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1812823008 Cr-Commit-Position: refs/heads/master@{#34953}
-
mlippautz authored
Now that we have page-local remembered sets (due to refilling in page granularity) we can perform all updates on the sets during compaction in parallel without caching slots locally. BUG=chromium:524425 LOG=N NOTRY=true Review URL: https://codereview.chromium.org/1811573002 Cr-Commit-Position: refs/heads/master@{#34952}
-
bbudge authored
LOG=N BUG=v8:4124 Review URL: https://codereview.chromium.org/1706053002 Cr-Commit-Position: refs/heads/master@{#34951}
-
ssanfilippo authored
Previous to this change, the dummy Return inside InterpreterAssembler::Abort caused TurboFan to emit incorrect code for handlers that made use of this call. The stack pointer would not be incremented before tail calling into the next handler, causing it to push on top on the caller's frame instead of overwriting it. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1819853002 Cr-Commit-Position: refs/heads/master@{#34950}
-
mlippautz authored
NOTRY=true BUG= Review URL: https://codereview.chromium.org/1817063002 Cr-Commit-Position: refs/heads/master@{#34949}
-
jkummerow authored
Bounds check hoisting was known to be buggy and has never been turned on. Since Crankshaft is deprecated, nobody is going to spend time fixing it, so let's just get rid of it. BUG=v8:4155,v8:4849 LOG=n R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1823623002 Cr-Commit-Position: refs/heads/master@{#34948}
-
mstarzinger authored
Note that this optimization only kicks in with --harmony-instanceof being enabled as well. By itself this optimization itself does not trigger. R=bmeurer@chromium.org BUG=v8:4447 LOG=n Review URL: https://codereview.chromium.org/1817043002 Cr-Commit-Position: refs/heads/master@{#34947}
-
machenbach authored
BUG=chromium:568949 LOG=n TBR=kjellander@chromium.org, tandrii@chromium.org NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1822703002 Cr-Commit-Position: refs/heads/master@{#34946}
-
yangguo authored
Code that we want to keep after warming up may have context-dependent inline caches. Clear these to avoid running into IC misses after deserialization. R=vogelheim@chromium.org BUG=v8:4836 LOG=N Review URL: https://codereview.chromium.org/1811263002 Cr-Commit-Position: refs/heads/master@{#34945}
-
verwaest authored
BUG= Committed: https://crrev.com/b3bda50ddd9e30b718ae7972d9dd0d095113f634 Cr-Commit-Position: refs/heads/master@{#34892} Review URL: https://codereview.chromium.org/1810083003 Cr-Commit-Position: refs/heads/master@{#34944}
-
machenbach authored
BUG=v8:2999,v8:4751 LOG=n NOTRY=true TBR=hablich@chromium.org Review URL: https://codereview.chromium.org/1820813003 Cr-Commit-Position: refs/heads/master@{#34943}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1815663002 Cr-Commit-Position: refs/heads/master@{#34942}
-
hlopko authored
BUG= LOG=no Review URL: https://codereview.chromium.org/1812203002 Cr-Commit-Position: refs/heads/master@{#34941}
-
mstarzinger authored
This rebaselines all our internal tests for error messages thrown by the implementation of 'instanceof' to the new ES6 semantics. It also applies a minor rephrasing to the messages in question. R=rossberg@chromium.org BUG=v8:4447 LOG=n Review URL: https://codereview.chromium.org/1822663002 Cr-Commit-Position: refs/heads/master@{#34940}
-
rmcilroy authored
The BytecodeArray shouldn't have changed since it was last loaded from the stack frame, so this store is unecessary. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1807673009 Cr-Commit-Position: refs/heads/master@{#34939}
-
yangguo authored
R=jochen@chromium.org BUG=chromium:595634 LOG=N Review URL: https://codereview.chromium.org/1820823003 Cr-Commit-Position: refs/heads/master@{#34938}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1820843002 Cr-Commit-Position: refs/heads/master@{#34937}
-
ulan authored
Currently we iterate the string table to clear non-live references and to update pointers. We can skip the second iteration if we record slots. BUG= Review URL: https://codereview.chromium.org/1814033004 Cr-Commit-Position: refs/heads/master@{#34936}
-
ishell authored
NOTRY=true Review URL: https://codereview.chromium.org/1822683002 Cr-Commit-Position: refs/heads/master@{#34935}
-
hpayer authored
BUG=596057 LOG=n Review URL: https://codereview.chromium.org/1818953002 Cr-Commit-Position: refs/heads/master@{#34934}
-
mstarzinger authored
R=rmcilroy@chromium.org TEST=cctest/test-interpreter/InterpreterInstanceOf BUG=v8:4447 LOG=n Review URL: https://codereview.chromium.org/1816063002 Cr-Commit-Position: refs/heads/master@{#34933}
-
jarin authored
Review URL: https://codereview.chromium.org/1820703003 Cr-Commit-Position: refs/heads/master@{#34932}
-
epertoso authored
BUG= Review URL: https://codereview.chromium.org/1808733002 Cr-Commit-Position: refs/heads/master@{#34931}
-
mvstanton authored
This CL restores special crankshaft optimizations for instance of when --harmony-instanceof is true. Similar work has to happen for TurboFan. I've added a flag to pile-up the optimizations to be done elsewhere, which will follow over several CLs. R=verwaest@chromium.org, mstarzinger@chromium.org BUG=v8:4447 LOG=n Review URL: https://codereview.chromium.org/1809113002 Cr-Commit-Position: refs/heads/master@{#34930}
-
ishell authored
TBR=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1817993002 Cr-Commit-Position: refs/heads/master@{#34929}
-
machenbach authored
NOTRY=true Review URL: https://codereview.chromium.org/1820853002 Cr-Commit-Position: refs/heads/master@{#34928}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1817913003 Cr-Commit-Position: refs/heads/master@{#34927}
-
Benedikt Meurer authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1819823002 . Cr-Commit-Position: refs/heads/master@{#34926}
-
epertoso authored
This is in preparation for a CL that does the equivalent of http://crrev.com/1780193003 for ia32. BUG= Review URL: https://codereview.chromium.org/1815213002 Cr-Commit-Position: refs/heads/master@{#34925}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1806283002 Cr-Commit-Position: refs/heads/master@{#34924}
-
machenbach authored
Unittest for feature added in: https://codereview.chromium.org/1808663002/ BUG=chromium:568949 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1810043004 Cr-Commit-Position: refs/heads/master@{#34923}
-
bmeurer authored
Split ToNumberStub into the entry ToNumberStub, and two new stubs, StringToNumberStub and NonNumberToNumberStub, which can be used when we already know something about the input (i.e. in various branches of the code stubs, or in TurboFan graphs). Also introduce an appropriate StringToNumber simplified operator for TurboFan, that is pure and is lowered to an invocation of the newly added StringToNumberStub. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1818923002 Cr-Commit-Position: refs/heads/master@{#34922}
-
machenbach authored
Rolling v8/build/gyp to 133eaf70b3720b7413017ef3f9c42f6811ef9347 Rolling v8/third_party/icu to 0d572d65aae621e13d6863fe470c9c8cee71043d This is the same as the auto-generated version, but with a landmine: https://codereview.chromium.org/1824563002/ TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Committed: https://crrev.com/1d6dba43c2b61900be68777297634e05d479a25a Cr-Commit-Position: refs/heads/master@{#34914} patch from issue 1824563002 at patchset 1 (http://crrev.com/1824563002#ps1) BUG= Review URL: https://codereview.chromium.org/1820823002 Cr-Commit-Position: refs/heads/master@{#34921}
-