[v8-dev] Issue 4208 in v8: Problems with very large switch-cases

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4208 by erik1...@gmail.com: Problems with very large switch-cases https://code.google.com/p/v8/issues/detail?id=4208 Version: latest commit on git master: 86594b7da24cce04f0801d8033029414433fb4ed OS: Ubuntu 15.04 x64 Architecture: x64 What steps will

[v8-dev] Issue 4211 in v8: Wrong precedence for arrow function without parameters

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4211 by andrebar...@googlemail.com: Wrong precedence for arrow function without parameters https://code.google.com/p/v8/issues/detail?id=4211 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Re: Issue 4211 in v8: Wrong precedence for arrow function without parameters

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: wi...@igalia.com Cc: a...@chromium.org rossb...@chromium.org Labels: Type-Bug Priority-Medium Area-Language HW-All OS-All Harmony Comment #1 on issue 4211 by a...@chromium.org: Wrong precedence for arrow function without

[v8-dev] Re: Issue 4217 in v8: |new| should construct objects from function instead of caller realm

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: rossb...@chromium.org Cc: a...@chromium.org Labels: Type-Bug Priority-Medium HW-All OS-All Comment #1 on issue 4217 by a...@chromium.org: |new| should construct objects from function instead of caller realm

[v8-dev] Re: Issue 4217 in v8: |new| should construct objects from function instead of caller realm

2015-06-22 Thread codesite-noreply via v8-dev
Comment #2 on issue 4217 by andrebar...@googlemail.com: |new| should construct objects from function instead of caller realm https://code.google.com/p/v8/issues/detail?id=4217 This is observable using iframes, test case from the linked bugs.ecmascript.org report: --- !doctype html html

[v8-dev] Issue 4227 in v8: Wrong operation order in JSON.stringify

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4227 by andrebar...@googlemail.com: Wrong operation order in JSON.stringify https://code.google.com/p/v8/issues/detail?id=4227 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8 JSON.stringify(,

[v8-dev] Issue 4209 in v8: AdjustAmountOfExternalAllocatedMemory should try to trigger scavenges

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4209 by chalk...@gmail.com: AdjustAmountOfExternalAllocatedMemory should try to trigger scavenges https://code.google.com/p/v8/issues/detail?id=4209 See https://github.com/nodejs/io.js/issues/1671 and testcases at

[v8-dev] Issue 4219 in v8: Intl ToDateTimeOptions must not modify options object

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4219 by andrebar...@googlemail.com: Intl ToDateTimeOptions must not modify options object https://code.google.com/p/v8/issues/detail?id=4219 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8 var

[v8-dev] Issue 4228 in v8: ToString not called for Number wrappers in JSON.stringify

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4228 by andrebar...@googlemail.com: ToString not called for Number wrappers in JSON.stringify https://code.google.com/p/v8/issues/detail?id=4228 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Issue 4232 in v8: Unchecked assignment with plain object in PromiseDeferred (promise.js)

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4232 by andrebar...@googlemail.com: Unchecked assignment with plain object in PromiseDeferred (promise.js) https://code.google.com/p/v8/issues/detail?id=4232 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1.

[v8-dev] Re: Issue 4213 in v8: Bad error message for eval/arguments as arrow function name in strict mode

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: wi...@igalia.com Cc: rossb...@chromium.org a...@chromium.org caitpott...@gmail.com Labels: Type-Bug Priority-Medium Area-Language OS-All HW-All Harmony Comment #1 on issue 4213 by a...@chromium.org: Bad error message for

[v8-dev] Issue 4216 in v8: Invalid default locale not handled in Intl

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4216 by andrebar...@googlemail.com: Invalid default locale not handled in Intl https://code.google.com/p/v8/issues/detail?id=4216 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Start d8 with bogus locale

[v8-dev] Issue 4218 in v8: ToLength called multiple times in %TypedArray%.prototype.set

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4218 by andrebar...@googlemail.com: ToLength called multiple times in %TypedArray%.prototype.set https://code.google.com/p/v8/issues/detail?id=4218 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8

[v8-dev] Issue 4225 in v8: Unchecked use of Date.prototype.toString in Date function

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4225 by andrebar...@googlemail.com: Unchecked use of Date.prototype.toString in Date function https://code.google.com/p/v8/issues/detail?id=4225 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Issue 4231 in v8: Don't allow try-catch variable redeclaration in for-of statement

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4231 by andrebar...@googlemail.com: Don't allow try-catch variable redeclaration in for-of statement https://code.google.com/p/v8/issues/detail?id=4231 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in

[v8-dev] Re: Fix wrong DCHECK in Heap::FindAllocationMemento where bump pointer overflow points to the currently… (issue 1200833003 by hpa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/3612) https://codereview.chromium.org/1200833003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev ---

[v8-dev] Let GC select the collector when the external memory allocation limit is reached (issue 1201993002 by joc...@chromium.org)

2015-06-22 Thread joc...@chromium.org via codereview.chromium.org
Reviewers: Hannes Payer, Description: Let GC select the collector when the external memory allocation limit is reached BUG=none R=hpa...@chromium.org LOG=n Please review this at https://codereview.chromium.org/1201993002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master

[v8-dev] Issue 4213 in v8: Bad error message for eval/arguments as arrow function name in strict mode

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4213 by andrebar...@googlemail.com: Bad error message for eval/arguments as arrow function name in strict mode https://code.google.com/p/v8/issues/detail?id=4213 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem?

[v8-dev] Re: Issue 4214 in v8: Wrong this-value when deleting property named eval in with-statement context

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: wi...@chromium.org Cc: a...@chromium.org rossb...@chromium.org Labels: Type-Bug Priority-Medium HW-All OS-All Area-Language Harmony Comment #1 on issue 4214 by a...@chromium.org: Wrong this-value when deleting property named eval

[v8-dev] Issue 4215 in v8: Duplicate variant and singleton subtags not detected if in different case

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4215 by andrebar...@googlemail.com: Duplicate variant and singleton subtags not detected if in different case https://code.google.com/p/v8/issues/detail?id=4215 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem?

[v8-dev] Merge AddFastElement and AddFastDoubleElement (issue 1198343004 by verwa...@chromium.org)

2015-06-22 Thread verwaest
Reviewers: Jakob, Message: ptal Description: Merge AddFastElement and AddFastDoubleElement BUG=v8:4137 LOG=n Please review this at https://codereview.chromium.org/1198343004/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+110, -259 lines): M src/builtins.cc

[v8-dev] Issue 4220 in v8: Unchecked use of built-ins in String.prototype.normalize

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4220 by andrebar...@googlemail.com: Unchecked use of built-ins in String.prototype.normalize https://code.google.com/p/v8/issues/detail?id=4220 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Re: Add support for multiple Workers in d8 (issue 1192923002 by bi...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 2 (id:??) landed as https://crrev.com/001ee86e323a722fd4c0761d0b1a2c3a0c8f8e4b Cr-Commit-Position: refs/heads/master@{#29195} https://codereview.chromium.org/1192923002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Inline SetLengthWithoutNormalize into its callers (issue 1194943002 by verwa...@chromium.org)

2015-06-22 Thread brucedawson
https://codereview.chromium.org/1194943002/diff/20001/src/elements.cc File src/elements.cc (right): https://codereview.chromium.org/1194943002/diff/20001/src/elements.cc#newcode1323 src/elements.cc:1323: Isolate* isolate = array-GetIsolate(); This declaration of 'isolate' shadows a seemingly

[v8-dev] Issue 4226 in v8: Unchecked use of built-ins in Date.prototype.toISOString

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4226 by andrebar...@googlemail.com: Unchecked use of built-ins in Date.prototype.toISOString https://code.google.com/p/v8/issues/detail?id=4226 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Fix wrong DCHECK in Heap::FindAllocationMemento where bump pointer overflow points to the currently… (issue 1200833003 by hpa...@chromium.org)

2015-06-22 Thread mvstanton
lgtm https://codereview.chromium.org/1200833003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop receiving emails from

[v8-dev] Re: Fix wrong DCHECK in Heap::FindAllocationMemento where bump pointer overflow points to the currently… (issue 1200833003 by hpa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1200833003/1 https://codereview.chromium.org/1200833003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Issue 4230 in v8: Function constructor rejects ( in parameters argument

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4230 by andrebar...@googlemail.com: Function constructor rejects ( in parameters argument https://code.google.com/p/v8/issues/detail?id=4230 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8

[v8-dev] Issue 4210 in v8: TDZ is not enforced in for-in/of loop expression

2015-06-22 Thread codesite-noreply via v8-dev
Status: Assigned Owner: ad...@chromium.org CC: a...@chromium.org, little...@chromium.org Labels: Type-Bug Priority-Medium Area-Language OS-All HW-All Harmony New issue 4210 by a...@chromium.org: TDZ is not enforced in for-in/of loop expression

[v8-dev] Issue 4212 in v8: yield allowed as parameter name in arrow function in generator context

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4212 by andrebar...@googlemail.com: yield allowed as parameter name in arrow function in generator context https://code.google.com/p/v8/issues/detail?id=4212 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1.

[v8-dev] Re: Issue 4212 in v8: yield allowed as parameter name in arrow function in generator context

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: wi...@igalia.com Cc: a...@chromium.org rossb...@chromium.org Labels: Type-Bug Priority-Medium Area-Language HW-All OS-All Harmony Comment #1 on issue 4212 by a...@chromium.org: yield allowed as parameter name in arrow function in

[v8-dev] Re: Add support for multiple Workers in d8 (issue 1192923002 by bi...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1192923002/40001 https://codereview.chromium.org/1192923002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Re: [Test262-es6] Update to use FAIL_SLOPPY everywhere (issue 1199013002 by a...@chromium.org)

2015-06-22 Thread arv
PTAL I had to expand the wildcards since now the outcome is enforced. https://codereview.chromium.org/1199013002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group.

[v8-dev] Re: Move SetFastElementsCapacity into GrowCapacityAndConvert (issue 1197133003 by verwa...@chromium.org)

2015-06-22 Thread jkummerow
LGTM with comments. https://codereview.chromium.org/1197133003/diff/60001/src/objects.cc File src/objects.cc (right): https://codereview.chromium.org/1197133003/diff/60001/src/objects.cc#newcode12428 src/objects.cc:12428: DCHECK(!(value-IsSmi() object-HasFastDoubleElements())); AFAICS you can

[v8-dev] Issue 4224 in v8: Unchecked use of String.prototype.charCodeAt in encode/decodeURI functions

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4224 by andrebar...@googlemail.com: Unchecked use of String.prototype.charCodeAt in encode/decodeURI functions https://code.google.com/p/v8/issues/detail?id=4224 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem?

[v8-dev] Issue 4229 in v8: Object.prototype.hasOwnProperty steps not executed in correct order

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4229 by andrebar...@googlemail.com: Object.prototype.hasOwnProperty steps not executed in correct order https://code.google.com/p/v8/issues/detail?id=4229 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run

[v8-dev] Issue 4217 in v8: |new| should construct objects from function instead of caller realm

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4217 by andrebar...@googlemail.com: |new| should construct objects from function instead of caller realm https://code.google.com/p/v8/issues/detail?id=4217 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run

[v8-dev] Re: Issue 4218 in v8: ToLength called multiple times in %TypedArray%.prototype.set

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Assigned Owner: little...@chromium.org Cc: ad...@chromium.org a...@chromium.org Labels: Type-Bug Priority-Medium HW-All OS-All Area-Language Harmony Comment #1 on issue 4218 by a...@chromium.org: ToLength called multiple times in

[v8-dev] Re: Issue 2222 in v8: Drop the escaped reserved words as identifiers compatibility measure

2015-06-22 Thread codesite-noreply via v8-dev
Comment #13 on issue by m...@bocoup.com: Drop the escaped reserved words as identifiers compatibility measure https://code.google.com/p/v8/issues/detail?id= This bug also leads to strange behavior for the case of the `this` keyword specifically: $ d8 -e var this;

[v8-dev] [Test262-es6] Update to use FAIL_SLOPPY everywhere (issue 1199013002 by a...@chromium.org)

2015-06-22 Thread arv
Reviewers: adamk, littledan, Message: PTAL I had to expand the wildcards since now the outcome is enforced. Description: [Test262-es6] Update to use FAIL_SLOPPY everywhere Also reordered related test BUG=N LOG=N R=adamk, littledan Please review this at

[v8-dev] Re: Issue 4208 in v8: Problems with very large switch-cases

2015-06-22 Thread codesite-noreply via v8-dev
Comment #1 on issue 4208 by erik1...@gmail.com: Problems with very large switch-cases https://code.google.com/p/v8/issues/detail?id=4208 Looks like i didn't upload the files the first time. Attachments: regress-80981.js 212 KB regress-324650.js 122 KB -- You received this

[v8-dev] Issue 4221 in v8: Unchecked use of String.prototype.replace in HtmlEscape (string.js)

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4221 by andrebar...@googlemail.com: Unchecked use of String.prototype.replace in HtmlEscape (string.js) https://code.google.com/p/v8/issues/detail?id=4221 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run

[v8-dev] Issue 4223 in v8: Incorrect operation order in Array.prototype.reverse

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4223 by andrebar...@googlemail.com: Incorrect operation order in Array.prototype.reverse https://code.google.com/p/v8/issues/detail?id=4223 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the problem? 1. Run in d8 d8 1 in

[v8-dev] Revert Revert relanded strong property access CL (issue 1199983002 by conr...@chromium.org)

2015-06-22 Thread conradw
Reviewers: Toon Verwaest, mvstanton, rossberg, Message: PTAL, regressions should be fixed. It may be worth waiting until after the branch cut to land this, however. Description: Revert Revert relanded strong property access CL Regression issues should be solved. Initial patchset is the

[v8-dev] Re: Issue 4209 in v8: AdjustAmountOfExternalAllocatedMemory should try to trigger scavenges

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Owner: joc...@chromium.org Comment #1 on issue 4209 by joc...@chromium.org: AdjustAmountOfExternalAllocatedMemory should try to trigger scavenges https://code.google.com/p/v8/issues/detail?id=4209 (No comment was entered for this change.) -- You received this message

[v8-dev] Re: Issue 4210 in v8: TDZ is not enforced in for-in/of loop expression

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Cc: rossb...@chromium.org Comment #1 on issue 4210 by a...@chromium.org: TDZ is not enforced in for-in/of loop expression https://code.google.com/p/v8/issues/detail?id=4210 (No comment was entered for this change.) -- You received this message because this project is

[v8-dev] Issue 4214 in v8: Wrong this-value when deleting property named eval in with-statement context

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4214 by andrebar...@googlemail.com: Wrong this-value when deleting property named eval in with-statement context https://code.google.com/p/v8/issues/detail?id=4214 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the

[v8-dev] Issue 4222 in v8: String.prototype html legacy methods need to use ToString to convert this-value

2015-06-22 Thread codesite-noreply via v8-dev
Status: Untriaged Owner: New issue 4222 by andrebar...@googlemail.com: String.prototype html legacy methods need to use ToString to convert this-value https://code.google.com/p/v8/issues/detail?id=4222 Version: 7142b0d211b732e1c119fded80f43fbbd9cea0f8 What steps will reproduce the

[v8-dev] Re: Add support for multiple Workers in d8 (issue 1192923002 by bi...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #2 (id:40001) https://codereview.chromium.org/1192923002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and

[v8-dev] [turbofan] Avoid embedding type feedback vector into code. (issue 1198263004 by mstarzin...@chromium.org)

2015-06-22 Thread mstarzinger
Reviewers: Benedikt Meurer, mvstanton, Message: Benedikt: PTAL. Michael: FYI. https://codereview.chromium.org/1198263004/diff/1/src/compiler/ast-graph-builder.h File src/compiler/ast-graph-builder.h (right):

[v8-dev] Re: Test262-es6 test runner should handle sloppy fail better (issue 1197913002 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #2 (id:20001) https://codereview.chromium.org/1197913002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and

[v8-dev] Fix string HTML methods to call ToString (issue 1194173004 by a...@chromium.org)

2015-06-22 Thread arv
Reviewers: adamk, littledan, Message: PTAL Description: Fix string HTML methods to call ToString Before this we were using + which calls valueOf which is not correct for these methods. BUG=v8:4222 LOG=N R=adamk, littledan Please review this at https://codereview.chromium.org/1194173004/

[v8-dev] Disable regress-1132 on ASAN runs (issue 1203523002 by little...@chromium.org)

2015-06-22 Thread littledan
Reviewers: adamk, Description: Disable regress-1132 on ASAN runs This test appears to trigger some bug in either ASAN or V8 when accompanied by an increase in JS code size. Disabling the test on ASAN runs to unblock adding new JS code. BUG=v8:4236 LOG=Y R=adamk Please review this at

[v8-dev] Use CHECK_LT in CheckHandleCountVisitor for better error message (issue 1201773005 by ad...@chromium.org)

2015-06-22 Thread adamk
Reviewers: arv, Description: Use CHECK_LT in CheckHandleCountVisitor for better error message Please review this at https://codereview.chromium.org/1201773005/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+1, -1 lines): M src/heap/heap.cc Index:

[v8-dev] Re: [Test262-es6] Update to use FAIL_SLOPPY everywhere (issue 1199013002 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #1 (id:1) https://codereview.chromium.org/1199013002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop

[v8-dev] Re: Fix wrong DCHECK in Heap::FindAllocationMemento where bump pointer overflow points to the currently… (issue 1200833003 by hpa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/3625) https://codereview.chromium.org/1200833003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev ---

[v8-dev] Re: [Test262-es6] Update to use FAIL_SLOPPY everywhere (issue 1199013002 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/7539f322eed1e9d8dd9d857eeba415949584d7cc Cr-Commit-Position: refs/heads/master@{#29200} https://codereview.chromium.org/1199013002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Fix HTML string methods to not depend on replace method (issue 1199933005 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/67943) https://codereview.chromium.org/1199933005/ -- -- v8-dev mailing list v8-dev@googlegroups.com

[v8-dev] Re: Remove duplicate isolate (issue 1196533004 by verwa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #1 (id:1) https://codereview.chromium.org/1196533004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop

[v8-dev] Re: Remove duplicate isolate (issue 1196533004 by verwa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/55a6a4977799f3c896d94db735f86be0052a8796 Cr-Commit-Position: refs/heads/master@{#29204} https://codereview.chromium.org/1196533004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Test262-es6 test runner should handle sloppy fail better (issue 1197913002 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1197913002/20001 https://codereview.chromium.org/1197913002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Re: Issue 4192 in v8: 'Hello World' sample segfaults on x64 linux

2015-06-22 Thread codesite-noreply via v8-dev
Comment #3 on issue 4192 by hed...@gmail.com: 'Hello World' sample segfaults on x64 linux https://code.google.com/p/v8/issues/detail?id=4192 Same issue here, same stack trace from gdb -- You received this message because this project is configured to send all issue notifications to this

[v8-dev] Re: Test262-es6 test runner should handle sloppy fail better (issue 1197913002 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 2 (id:??) landed as https://crrev.com/9f550240b03f2894ec97d6b304d1445916b9b26c Cr-Commit-Position: refs/heads/master@{#29194} https://codereview.chromium.org/1197913002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Issue 4164 in v8: Make the status file support filtering on variants or flags

2015-06-22 Thread codesite-noreply via v8-dev
Comment #7 on issue 4164 by bugdro...@chromium.org: Make the status file support filtering on variants or flags https://code.google.com/p/v8/issues/detail?id=4164#c7 The following revision refers to this bug:

[v8-dev] PPC: Built-in apply() performance benefits from an uninitialized IC. (issue 1196253003 by mbra...@us.ibm.com)

2015-06-22 Thread mbrandy
Reviewers: dstence, michael_dawson, mvstanton, Description: PPC: Built-in apply() performance benefits from an uninitialized IC. Port 2a3b05758721a2f63a7c611ef22e364ead9ed0f3 R=mvstan...@chromium.org, dste...@us.ibm.com, michael_daw...@ca.ibm.com BUG= Please review this at

[v8-dev] PPC: Vector ICs: Turbofan vector store ic support (issue 1201983005 by mbra...@us.ibm.com)

2015-06-22 Thread mbrandy
Reviewers: dstence, michael_dawson, mvstanton, Description: PPC: Vector ICs: Turbofan vector store ic support Port 17c8ffeaa3e4e1769f0b4d0e7c8a6fb26b5297bd Original commit message: Turbofan needs to pass vector slots around for named and keyed stores. Also, the CL addresses a missing slot for

[v8-dev] Re: Issue 4222 in v8: String.prototype html legacy methods need to use ToString to convert this-value

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Started Cc: rossb...@chromium.org Labels: Type-Bug Priority-Medium HW-All OS-All Comment #2 on issue 4222 by a...@chromium.org: String.prototype html legacy methods need to use ToString to convert this-value

[v8-dev] Re: [Test262-es6] Update to use FAIL_SLOPPY everywhere (issue 1199013002 by a...@chromium.org)

2015-06-22 Thread littledan
lgtm https://codereview.chromium.org/1199013002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop receiving emails from

[v8-dev] Re: Issue 4221 in v8: Unchecked use of String.prototype.replace in HtmlEscape (string.js)

2015-06-22 Thread codesite-noreply via v8-dev
Comment #2 on issue 4221 by a...@chromium.org: Unchecked use of String.prototype.replace in HtmlEscape (string.js) https://code.google.com/p/v8/issues/detail?id=4221 https://codereview.chromium.org/1199933005 -- You received this message because this project is configured to send all

[v8-dev] Re: PPC: Built-in apply() performance benefits from an uninitialized IC. (issue 1196253003 by mbra...@us.ibm.com)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/40ec8e1db0bfd30f8c212c51a56bf2caed5e261d Cr-Commit-Position: refs/heads/master@{#29196} https://codereview.chromium.org/1196253003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Remove duplicate isolate (issue 1196533004 by verwa...@chromium.org)

2015-06-22 Thread verwaest
Reviewers: Jakob, brucedawson, Message: ptal Description: Remove duplicate isolate BUG= Please review this at https://codereview.chromium.org/1196533004/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+0, -1 lines): M src/elements.cc Index: src/elements.cc

[v8-dev] Re: Inline SetLengthWithoutNormalize into its callers (issue 1194943002 by verwa...@chromium.org)

2015-06-22 Thread verwaest
Will do so: https://codereview.chromium.org/1196533004 https://codereview.chromium.org/1194943002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe

[v8-dev] Re: Fix wrong DCHECK in Heap::FindAllocationMemento where bump pointer overflow points to the currently… (issue 1200833003 by hpa...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1200833003/1 https://codereview.chromium.org/1200833003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Re: Remove duplicate isolate (issue 1196533004 by verwa...@chromium.org)

2015-06-22 Thread jkummerow
lgtm https://codereview.chromium.org/1196533004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop receiving emails from

[v8-dev] Re: Issue 4236 in v8: mjsunit/regress-1132 is unstable on ASAN

2015-06-22 Thread codesite-noreply via v8-dev
Comment #1 on issue 4236 by bugdro...@chromium.org: mjsunit/regress-1132 is unstable on ASAN https://code.google.com/p/v8/issues/detail?id=4236#c1 The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/7ebf6fc965071dde324a12f40442c17955d8275a commit

[v8-dev] Re: Issue 4222 in v8: String.prototype html legacy methods need to use ToString to convert this-value

2015-06-22 Thread codesite-noreply via v8-dev
Comment #4 on issue 4222 by bugdro...@chromium.org: String.prototype html legacy methods need to use ToString to convert this-value https://code.google.com/p/v8/issues/detail?id=4222#c4 The following revision refers to this bug:

[v8-dev] Re: Issue 4234 in v8: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in)

2015-06-22 Thread codesite-noreply via v8-dev
Comment #4 on issue 4234 by andrebar...@googlemail.com: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in) https://code.google.com/p/v8/issues/detail?id=4234 9.2.1.2 OrdinaryCallBindThis ( F, calleeContext, thisArgument ), step 6.b is the relevant spec algorithm for this

[v8-dev] Re: Fix string HTML methods to call ToString (issue 1194173004 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/6b268bce6a835b0517468b4b4253b7ce2affdad8 Cr-Commit-Position: refs/heads/master@{#29202} https://codereview.chromium.org/1194173004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Use CHECK_LT in CheckHandleCountVisitor for better error message (issue 1201773005 by ad...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/97a887c530da3252d85ab666b125d9ff0faed790 Cr-Commit-Position: refs/heads/master@{#29203} https://codereview.chromium.org/1201773005/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: Use CHECK_LT in CheckHandleCountVisitor for better error message (issue 1201773005 by ad...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #1 (id:1) https://codereview.chromium.org/1201773005/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from this group and stop

[v8-dev] Re: Issue 4222 in v8: String.prototype html legacy methods need to use ToString to convert this-value

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Fixed Comment #5 on issue 4222 by a...@chromium.org: String.prototype html legacy methods need to use ToString to convert this-value https://code.google.com/p/v8/issues/detail?id=4222 (No comment was entered for this change.) -- You received this message because

[v8-dev] Re: Fix HTML string methods to not depend on replace method (issue 1199933005 by a...@chromium.org)

2015-06-22 Thread arv
Reviewers: adamk, littledan, Message: On 2015/06/22 19:40:02, commit-bot: I haz the power wrote: Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/67943) Needs git rebase

[v8-dev] Re: Fix HTML string methods to not depend on replace method (issue 1199933005 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1199933005/20001 https://codereview.chromium.org/1199933005/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Issue 4237 in v8: mjsunit/regress-3976 is unstable under the Harmony TypedArray patch

2015-06-22 Thread codesite-noreply via v8-dev
Status: Assigned Owner: erikco...@chromium.org Labels: Type-Bug Priority-Medium New issue 4237 by little...@chromium.org: mjsunit/regress-3976 is unstable under the Harmony TypedArray patch https://code.google.com/p/v8/issues/detail?id=4237 This test leads to a crash when run with the

[v8-dev] Re: Issue 4235 in v8: Template strings do not work in v8 natives

2015-06-22 Thread codesite-noreply via v8-dev
Comment #1 on issue 4235 by a...@chromium.org: Template strings do not work in v8 natives https://code.google.com/p/v8/issues/detail?id=4235 I see it being used in v8natives.js so not sure why it would not work in strings.js -- You received this message because this project is configured

[v8-dev] Re: Issue 4234 in v8: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in)

2015-06-22 Thread codesite-noreply via v8-dev
Comment #6 on issue 4234 by ad...@chromium.org: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in) https://code.google.com/p/v8/issues/detail?id=4234 The easiest way to fix this would be to expose Builtins::kFunctionCall as $functionCall in the natives JS. That builtin takes

[v8-dev] Re: Add an informative comment on regress-1132 ASAN suppression (issue 1199053002 by ad...@chromium.org)

2015-06-22 Thread adamk
On 2015/06/22 21:26:46, arv wrote: LGTM A url would have been better than just a bug number. I was going for consistency with the file, but clearly that's not great, sorry. https://codereview.chromium.org/1199053002/ -- -- v8-dev mailing list v8-dev@googlegroups.com

[v8-dev] Re: Issue 4234 in v8: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in)

2015-06-22 Thread codesite-noreply via v8-dev
Comment #8 on issue 4234 by verwa...@chromium.org: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in) https://code.google.com/p/v8/issues/detail?id=4234 The latter we could fix by just replacing null with undefined in case of sloppy mode callee though... Its prologue will

[v8-dev] Remove usage of S.p.charCodeAt from uri.js (issue 1204483003 by a...@chromium.org)

2015-06-22 Thread arv
Reviewers: adamk, littledan, Message: PTAL https://codereview.chromium.org/1204483003/diff/1/src/uri.js File src/uri.js (right): https://codereview.chromium.org/1204483003/diff/1/src/uri.js#newcode167 src/uri.js:167: uri = TO_STRING_INLINE(uri); The spec calls this as a first step. This is

[v8-dev] Re: Fix HTML string methods to not depend on replace method (issue 1199933005 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Try jobs failed on following builders: v8_linux_nodcheck_rel on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel/builds/3870) https://codereview.chromium.org/1199933005/ -- -- v8-dev mailing list v8-dev@googlegroups.com

[v8-dev] Re: Issue 4234 in v8: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in)

2015-06-22 Thread codesite-noreply via v8-dev
Comment #5 on issue 4234 by ad...@chromium.org: Wrapper created from wrong realm (SHOULD_CREATE_WRAPPER built-in) https://code.google.com/p/v8/issues/detail?id=4234 Ah, sorry, I was just misunderstanding the test case (was thinking of the ToObject on the receiver of Array.prototype.map,

[v8-dev] Re: Fix HTML string methods to not depend on replace method (issue 1199933005 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1199933005/40001 https://codereview.chromium.org/1199933005/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Re: i18n.js was not using original functions (issue 1199813004 by a...@chromium.org)

2015-06-22 Thread adamk
lgtm +jochen FYI since he imported this code. https://codereview.chromium.org/1199813004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups v8-dev group. To unsubscribe from

[v8-dev] Add an informative comment on regress-1132 ASAN suppression (issue 1199053002 by ad...@chromium.org)

2015-06-22 Thread ad...@chromium.org via codereview.chromium.org
Reviewers: arv, Message: Committed patchset #1 (id:1) manually as 1c575e917a8f32ed516ed03459659755a35f1262 (presubmit successful). Description: Add an informative comment on regress-1132 ASAN suppression TBR=a...@chromium.org LOG=n Committed:

[v8-dev] Re: i18n.js was not using original functions (issue 1199813004 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
Try jobs failed on following builders: v8_linux_rel on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel/builds/6882) https://codereview.chromium.org/1199813004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev ---

[v8-dev] Cleanup adding elements and in particular dictionary elements (issue 1196163005 by verwa...@chromium.org)

2015-06-22 Thread verwaest
Reviewers: Jakob, Message: ptal Description: Cleanup adding elements and in particular dictionary elements BUG=v8:4137 LOG=n Please review this at https://codereview.chromium.org/1196163005/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+68, -129 lines): M

[v8-dev] i18n.js was not using original functions (issue 1199813004 by a...@chromium.org)

2015-06-22 Thread arv
Reviewers: adamk, littledan, Message: PTAL Description: i18n.js was not using original functions The i18n.js code was calling a lot of methods, which might have been removed or replaced by user code. Make sure we use the original functions. BUG=v8:4220 LOG=N R=adamk, littledan

[v8-dev] Re: i18n.js was not using original functions (issue 1199813004 by a...@chromium.org)

2015-06-22 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1199813004/20001 https://codereview.chromium.org/1199813004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are

[v8-dev] Also check for access checks and indexed interceptors before allowing fast moving of elements (issue 1200053002 by verwa...@chromium.org)

2015-06-22 Thread verwaest
Reviewers: mvstanton, Message: ptal Description: Also check for access checks and indexed interceptors before allowing fast moving of elements BUG= Please review this at https://codereview.chromium.org/1200053002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files

[v8-dev] Re: Issue 4233 in v8: Wrong operation order when evaluating super property

2015-06-22 Thread codesite-noreply via v8-dev
Updates: Status: Started Owner: a...@chromium.org Cc: rossb...@chromium.org dslo...@chromium.org Labels: Type-Bug Priority-Medium HW-All OS-All Harmony Area-Language Comment #1 on issue 4233 by a...@chromium.org: Wrong operation order when evaluating super

  1   2   3   4   >