| 2016-08-19 Benjamin Poulain <bpoulain@apple.com> |
| |
| [JSC] ArithSqrt should work with any argument type |
| https://bugs.webkit.org/show_bug.cgi?id=160954 |
| |
| Reviewed by Saam Barati. |
| |
| * stress/arith-sqrt-on-various-types.js: Added. |
| (let.validInputTypedTestCases.validInputTestCases.map): |
| (isIdentical): |
| (opaqueAllTypesSqrt): |
| (testAllTypesCall): |
| (testSingleTypeCall): |
| (opaqueSqrtForSideEffects): |
| (testSideEffect.let.testObject.valueOf): |
| (testSideEffect): |
| (opaqueSqrtForCSE): |
| (testCSE.let.testObject.valueOf): |
| (testCSE): |
| (testException.opaqueSqrtWithException): |
| (testException): |
| |
| 2016-08-19 Joseph Pecoraro <pecoraro@apple.com> |
| |
| Make custom Error properties (line, column, sourceURL) configurable and writable |
| https://bugs.webkit.org/show_bug.cgi?id=160984 |
| <rdar://problem/27905979> |
| |
| Reviewed by Saam Barati. |
| |
| * stress/native-error-properties.js: Added. |
| (assert): |
| (shouldNotThrow): |
| |
| (checkEmptyErrorPropertiesDescriptors): |
| (checkNonEmptyErrorPropertiesDescriptors): |
| The spec only describes the "message" property, so |
| ensure it has the right descriptor attributes. |
| |
| (checkErrorPropertiesWritable): |
| Ensure common error property names are writable. |
| In strict mode this would have thrown an exception |
| if they were readonly. |
| |
| 2016-08-18 Mark Lam <mark.lam@apple.com> |
| |
| ScopedArguments is using the wrong owner object for a write barrier. |
| https://bugs.webkit.org/show_bug.cgi?id=160976 |
| <rdar://problem/27328506> |
| |
| Reviewed by Keith Miller. |
| |
| * stress/scoped-arguments-write-barrier-should-be-on-scope-object.js: Added. |
| |
| 2016-08-17 JF Bastien <jfbastien@apple.com> |
| |
| We allow assignments to const variables when in a for-in/for-of loop |
| https://bugs.webkit.org/show_bug.cgi?id=156673 |
| |
| Reviewed by Filip Pizlo. |
| |
| * stress/for-in-of-const.js: Added. |
| (expect_nothrow): |
| (expect_throw): |
| (capture): |
| |
| 2016-08-17 Mark Lam <mark.lam@apple.com> |
| |
| Remove an invalid assertion in the DFG backend's GetById emitter. |
| https://bugs.webkit.org/show_bug.cgi?id=160925 |
| <rdar://problem/27248961> |
| |
| Reviewed by Filip Pizlo. |
| |
| * stress/dfg-get-by-id-should-not-assert-non-null-prediction.js: Added. |
| |
| 2016-08-16 Ryan Haddad <ryanhaddad@apple.com> |
| |
| Unreviewed, rolling out r204464. |
| |
| This is no longer needed after r204495. |
| |
| Reverted changeset: |
| |
| "Skip failing test mozilla/ecma/LexicalConventions/7.7.3.js" |
| https://bugs.webkit.org/show_bug.cgi?id=160662 |
| http://trac.webkit.org/changeset/204464 |
| |
| 2016-08-15 Ryan Haddad <ryanhaddad@apple.com> |
| |
| Skip failing test mozilla/ecma/LexicalConventions/7.7.3.js |
| https://bugs.webkit.org/show_bug.cgi?id=160662 |
| |
| Unreviewed test gardening. |
| |
| * mozilla/ecma/LexicalConventions/7.7.3.js: |
| |
| 2016-08-14 Skachkov Oleksandr <gskachkov@gmail.com> |
| |
| [2016] Set correct status for test262 after implementation of Object.values&Object.entries |
| https://bugs.webkit.org/show_bug.cgi?id=160844 |
| |
| Reviewed by Saam Barati. |
| |
| Patch contains fix statuses of specs in the test262 test collection after implementation of |
| Object.values and Object.entries functions. Also patch contains small fixes in tests of the |
| tests for Object.values/entries functions. |
| |
| * stress/object-entries.js: |
| (Object.getOwnPropertyDescriptor): |
| * stress/object-values.js: |
| (Object.getOwnPropertyDescriptor): |
| * test262.yaml: |
| |
| 2016-08-12 Saam Barati <sbarati@apple.com> |
| |
| Inline store loop for CopyRest in DFG and FTL for certain array modes |
| https://bugs.webkit.org/show_bug.cgi?id=159612 |
| |
| Reviewed by Filip Pizlo. |
| |
| * stress/rest-parameter-having-a-bad-time.js: Added. |
| * stress/rest-parameter-many-arguments.js: Added. |
| * stress/rest-parameter-various-types.js: Added. |
| |
| 2016-08-12 Skachkov Oleksandr <gskachkov@gmail.com> |
| |
| [ES2016] Implement Object.entries |
| https://bugs.webkit.org/show_bug.cgi?id=160412 |
| |
| Reviewed by Saam Barati. |
| |
| Patch contains tests for Object.entries function and |
| fix of wrong tests for Object.values function. |
| |
| * stress/object-entries.js: |
| (compare): |
| (string_appeared_here.forEach): |
| (const.getInvokedFunctions.): |
| (const.getInvokedFunctions): |
| (Array.prototype.push): |
| * stress/object-values.js: |
| |
| 2016-08-11 Mark Lam <mark.lam@apple.com> |
| |
| OverridesHasInstance should not branch across register allocations. |
| https://bugs.webkit.org/show_bug.cgi?id=160792 |
| <rdar://problem/27361778> |
| |
| Reviewed by Benjamin Poulain. |
| |
| * stress/OverrideHasInstance-should-not-branch-across-register-allocations.js: Added. |
| |
| 2016-08-11 Mark Lam <mark.lam@apple.com> |
| |
| The jsc shell's Element host constructor should throw if it fails to construct an object. |
| https://bugs.webkit.org/show_bug.cgi?id=160773 |
| <rdar://problem/27328608> |
| |
| Reviewed by Saam Barati. |
| |
| * stress/generational-opaque-roots.js: |
| |
| 2016-08-11 Mark Lam <mark.lam@apple.com> |
| |
| Disallow synchronous sweeping for eden GCs. |
| https://bugs.webkit.org/show_bug.cgi?id=160716 |
| |
| Reviewed by Geoffrey Garen. |
| |
| * stress/eden-gc-with-retired-blocks.js: Added. |
| - This test is just in case we add back support for eden GCs with synchronous |
| sweeping in the future. |
| |
| 2016-08-10 Michael Saboff <msaboff@apple.com> |
| |
| Baseline GetByVal and PutByVal for cache ID stubs need to handle exceptions |
| https://bugs.webkit.org/show_bug.cgi?id=160749 |
| |
| Reviewed by Filip Pizlo. |
| |
| New test that causes baseline GetByValWithCachedId and PutByValWithCachedId |
| stubs to be generated and then throws exceptions for those stub to handle |
| to verify that they are properly handled. |
| |
| * stress/regress-160749.js: Added. |
| (testCachedGetByVal.): |
| (testCachedGetByVal.get for): |
| (testCachedGetByVal): |
| (testCachedPutByVal.): |
| (testCachedPutByVal.set for): |
| (testCachedPutByVal): |
| |
| 2016-08-10 Mark Lam <mark.lam@apple.com> |
| |
| DFG's flushForTerminal() needs to add PhantomLocals for bytecode live locals. |
| https://bugs.webkit.org/show_bug.cgi?id=160755 |
| <rdar://problem/27488507> |
| |
| Reviewed by Filip Pizlo. |
| |
| * stress/need-bytecode-liveness-for-unreachable-blocks-at-dfg-time.js: Added. |
| |
| 2016-08-09 Skachkov Oleksandr <gskachkov@gmail.com> |
| |
| [ES2016] Implement Object.values |
| https://bugs.webkit.org/show_bug.cgi?id=160410 |
| |
| Reviewed by Saam Barati, Yusuke Suzuki. |
| |
| * stress/object-values.js: Added. |
| (compare): |
| (string_appeared_here.forEach): |
| (const.getInvokedFunctions.): |
| (const.getInvokedFunctions): |
| (Array.prototype.push): |
| |
| 2016-08-09 Saam Barati <sbarati@apple.com> |
| |
| JSBoundFunction should lazily generate its name string |
| https://bugs.webkit.org/show_bug.cgi?id=160678 |
| <rdar://problem/27043194> |
| |
| Reviewed by Mark Lam. |
| |
| * stress/bound-function-lazy-name-generation.js: Added. |
| (assert): |
| (test.let.f): |
| (test.f): |
| (test): |
| |
| 2016-08-08 Mark Lam <mark.lam@apple.com> |
| |
| ASSERTION FAILED: hasInlineStorage() in JSFinalObject::visitChildren(). |
| https://bugs.webkit.org/show_bug.cgi?id=160666 |
| |
| Reviewed by Keith Miller. |
| |
| * stress/object-constructor-should-be-new-target-aware.js: |
| |
| 2016-08-07 Yusuke Suzuki <utatane.tea@gmail.com> |
| |
| [ES6] Module namespace object should not allow unset IC |
| https://bugs.webkit.org/show_bug.cgi?id=160553 |
| |
| Reviewed by Saam Barati. |
| |
| * modules/namespace-object-get-property.js: Added. |
| (import.as.ns.from.string_appeared_here.shouldThrow): |
| * modules/namespace-object-has-property.js: Added. |
| * modules/namespace-object-inline-caching.js: Added. |
| (import.as.A.from.string_appeared_here.import.as.B.from.string_appeared_here.lookup): |
| (shouldBe.lookup.lookup): |
| (shouldBe.lookup): |
| * modules/namespace-object-inline-caching/a.js: Added. |
| * modules/namespace-object-inline-caching/b.js: Added. |
| * modules/namespace-object-try-get.js: Added. |
| (import.as.ns.from.string_appeared_here.tryGetByIdText): |
| (tryGetByIdTextStrict): |
| * modules/namespace-object-typed-array-fast-path.js: Added. |
| * test262.yaml: |
| |
| 2016-08-05 Saam Barati <sbarati@apple.com> |
| |
| various math operations don't properly check for an exception after calling toNumber() on the lhs |
| https://bugs.webkit.org/show_bug.cgi?id=160154 |
| |
| Reviewed by Mark Lam. |
| |
| * stress/to-number-throws-correct-exception.js: Added. |
| (test.let.test.runTest.): |
| (test.let.test.runTest.get f): |
| (test.let.test.runTest): |
| (test.let.test): |
| (test): |
| (test2.runTest.): |
| (test2.runTest.get f): |
| (test2.runTest): |
| (test2): |
| |
| 2016-08-05 Saam Barati <sbarati@apple.com> |
| |
| Assertion failure when accessing TDZ variable in catch through eval |
| https://bugs.webkit.org/show_bug.cgi?id=160554 |
| |
| Reviewed by Mark Lam and Keith Miller. |
| |
| * stress/catch-variables-under-tdz.js: Added. |
| (test): |
| |
| 2016-08-04 Yusuke Suzuki <utatane.tea@gmail.com> |
| |
| [ES6] JSModuleNamespaceObject's Symbol.iterator function should have name |
| https://bugs.webkit.org/show_bug.cgi?id=160549 |
| |
| Reviewed by Saam Barati. |
| |
| * modules/namespace-object-symbol-iterator-name.js: Added. |
| * test262.yaml: |
| |
| 2016-08-04 Keith Miller <keith_miller@apple.com> |
| |
| ASSERTION FAILED: !hasInstanceValueNode->isCellConstant() || defaultHasInstanceFunction == hasInstanceValueNode->asCell() |
| https://bugs.webkit.org/show_bug.cgi?id=160562 |
| |
| Reviewed by Mark Lam. |
| |
| * stress/instanceof-late-constant-folding.js: Added. |
| (Constructor): |
| (value): |
| (body): |
| |
| 2016-08-04 Caitlin Potter <caitp@igalia.com> |
| |
| [JSC] fix generator-syntax.js JSTest again after yield grammar fix |
| https://bugs.webkit.org/show_bug.cgi?id=160550 |
| |
| Reviewed by Yusuke Suzuki. |
| |
| * stress/generator-syntax.js: |
| (testYieldBindingIdentifier): |
| |
| 2016-08-03 Caitlin Potter <caitp@igalia.com> |
| |
| Clarify SyntaxErrors around yield and unskip tests |
| https://bugs.webkit.org/show_bug.cgi?id=158460 |
| |
| Reviewed by Saam Barati. |
| |
| Fix and unskip tests which erroneously asserted that `yield` is not a |
| valid BindingIdentifier, and improve error message for YieldExpressions |
| occuring in Arrow formal parameters. |
| |
| * stress/generator-syntax.js: |
| * stress/yield-out-of-generator.js: |
| |
| 2016-08-03 Filip Pizlo <fpizlo@apple.com> |
| |
| REGRESSION(r203368): broke some test262 tests |
| https://bugs.webkit.org/show_bug.cgi?id=160479 |
| |
| Reviewed by Mark Lam. |
| |
| Added a stress test for this case, since we don't always run test262. |
| |
| * stress/freeze-setter.js: Added. |
| (let.o.set foo): |
| |
| 2016-08-03 Saam Barati <sbarati@apple.com> |
| |
| Implement nested rest destructuring w.r.t the ES7 spec |
| https://bugs.webkit.org/show_bug.cgi?id=160423 |
| |
| Reviewed by Filip Pizlo. |
| |
| * stress/destructuring-rest-element.js: Added. |
| (assert): |
| (test): |
| (arr): |
| (eq): |
| (gen): |
| (fakeGen.return.Symbol.iterator): |
| (fakeGen): |
| * stress/rest-elements.js: |
| (testSyntaxError.String.raw): |
| * stress/rest-parameter-is-destructuring.js: Added. |
| (assert): |
| (test): |
| (foo): |
| (bar): |
| * test262.yaml: |
| |
| 2016-08-02 Saam Barati <sbarati@apple.com> |
| |
| Rename Changelog to ChangeLog |
| |
| Rubber stamped by Keith Miller. |
| |
| * Changelog: Removed. |
| |
| 2016-08-02 Saam Barati <sbarati@apple.com> |
| |
| update a class extending null w.r.t the ES7 spec |
| https://bugs.webkit.org/show_bug.cgi?id=160417 |
| |
| Reviewed by Keith Miller. |
| |
| * Changelog: |
| * stress/class-derived-from-null.js: Added. |
| (assert): |
| (test): |
| (test1.C): |
| (test1): |
| (jsNull): |
| (test2.C): |
| (test2): |
| (test3.C): |
| (test3.D): |
| (test3.E): |
| (test3): |
| (test4.E): |
| (test4): |
| (test5.E): |
| (test5): |
| (test6.Base): |
| (test6.D): |
| (test6.E): |
| (test6.F): |
| (test6.G): |
| (test6): |
| * test262.yaml: |
| |
| 2016-08-01 Filip Pizlo <fpizlo@apple.com> |
| |
| Rationalize varargs stack overflow checks |
| https://bugs.webkit.org/show_bug.cgi?id=160425 |
| |
| Reviewed by Michael Saboff. |
| |
| * stress/arity-check-ftl-throw-more-args.js: Added. |
| (catch): |
| |
| 2016-08-01 Keith Miller <keith_miller@apple.com> |
| |
| We should not keep the JavaScript tests inside the Source/JavaScriptCore/ directory. |
| https://bugs.webkit.org/show_bug.cgi?id=160372 |
| |
| Rubber stamped by Geoffrey Garen. |
| |
| This patch moves all the JavaScript tests from Source/JavaScriptCore/tests to |
| a new top level directory, JSTests. Having the tests in the Source directory |
| was both confusing an inconvenient for people that just want to checkout the |
| source code of WebKit. Since there is no other obvious place to put all the |
| JavaScript tests a new top level directory seemed the most sensible. |
| |
| * tests/: Deleted. |
