Updates:
Status: Duplicate
Mergedinto: chromium:515382
Comment #3 on issue 4332 by yang...@chromium.org: JSON.parse() fails
without helpful line number of line offset information when unexpected
token is encountered
https://code.google.com/p/v8/issues/detail?id=4332
(No comm
Comment #2 on issue 4332 by yang...@chromium.org: JSON.parse() fails
without helpful line number of line offset information when unexpected
token is encountered
https://code.google.com/p/v8/issues/detail?id=4332
This is a Chrome issue, not a V8 one. In d8, you would get this:
undefined:3:
https://codereview.chromium.org/1259203002/diff/140001/src/compiler/ast-graph-builder.cc
File src/compiler/ast-graph-builder.cc (right):
https://codereview.chromium.org/1259203002/diff/140001/src/compiler/ast-graph-builder.cc#newcode2606
src/compiler/ast-graph-builder.cc:2606: TailCallMode mode
This was not needed for interpreter, as callee-saved regs turned out to be
unnecessary there, so the urgency is removed. However, this does remove a
couple
instructions, and is probably a worthwhile cleanup. I will revisit this in
next
couple days, and add mips64, if I decide to land this.
h
This approach will not work because you add new static initializers until
uniform initialization and constexpr is allowed.
https://codereview.chromium.org/1266603002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because yo
Comment #6 on issue 2397 by yang...@chromium.org: Error.toString() throws
illegal access
https://code.google.com/p/v8/issues/detail?id=2397
"type" and "arguments" were special properties used internally by V8. That
was a bug that has been fixed. They are no longer used internally so it
sh
Awesome! LGTM with nit.
https://codereview.chromium.org/1262803002/diff/1/src/ast-numbering.cc
File src/ast-numbering.cc (right):
https://codereview.chromium.org/1262803002/diff/1/src/ast-numbering.cc#newcode326
src/ast-numbering.cc:326: if (expr->IsProperty())
Nit: if { ... } else { ... }
htt
lgtm
https://codereview.chromium.org/1260733003/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1266743002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1266743002/1
https://codereview.chromium.org/1266743002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
LGTM.
https://codereview.chromium.org/1263723002/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1265663002/20001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1265663002/20001
https://codereview.chromium.org/1265663002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
On 2015/07/30 06:01:35, Benedikt Meurer wrote:
Hey Yang,
Simple fix. Please take a look.
Thanks,
Benedikt
lgtm
https://codereview.chromium.org/1266743002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you a
Hey Yang,
Simple fix. Please take a look.
Thanks,
Benedikt
https://codereview.chromium.org/1266743002/
--
--
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 unsu
Reviewers: Yang,
Description:
[turbofan] Fix invalid access to Parameter index.
A Parameter operator contains a ParameterInfo payload, not an int
payload, so the OpParameter cast is invalid (and only worked by
coincidence currently).
BUG=chromium:515215
LOG=n
R=yang...@chromium.org
Please revi
OK, here is the updated mips32 port, and a mips64 version.
I tried to get too clever and and rebase this on unlanded
https://codereview.chromium.org/1254293006/ which is in the CQ. That didn't
work, so there an extra patchset to clean that up. You will have to fixup
linkage-mips[64].cc since this
Comment #6 on issue 4277 by seju...@gmail.com: Problem with
MOVW_MOVT_IMMEDIATE_LOADS feature
https://code.google.com/p/v8/issues/detail?id=4277
Hi ulan. The movw, movt instructions still cause assert fail error. The
expected size(28) and generated code size(24) are different each other at
Status: Untriaged
Owner:
New issue 4335 by rsturg...@google.com: large google maps performance
regression in chrome canary (version 46)
https://code.google.com/p/v8/issues/detail?id=4335
google maps in chrome 46 has some really bad framerate issues.
To repro just go to maps.google.com (
Comment #5 on issue 2397 by a...@goodeggs.com: Error.toString() throws
illegal access
https://code.google.com/p/v8/issues/detail?id=2397
Thanks for fixing. Could you speak a little more to "'type'
and 'arguments' properties should not be set for Error objects"? It
doesn't seem to be wid
On 2015/07/29 20:45:06, ofrobots wrote:
On 2015/07/24 06:32:10, Hannes Payer wrote:
> https://codereview.chromium.org/1252053003/diff/1/src/heap/spaces.cc
> File src/heap/spaces.cc (right):
>
>
https://codereview.chromium.org/1252053003/diff/1/src/heap/spaces.cc#newcode1478
> src/heap/spaces.c
Reviewers: Hannes Payer,
Description:
remove recursion from NewSpace::AllocateRaw*
The recursion between AllocateRaw* and SlowAllocateRaw makes incremental
stepping very complicated. This patch removes the recursion. Follow-on
patches will improve accounting of allocations done by incremental ma
Comment #11 on issue 3915 by waldron@gmail.com: Implement
exponentiation operator
https://code.google.com/p/v8/issues/detail?id=3915
Here's the latest specification
http://rwaldron.github.io/exponentiation-operator/
--
You received this message because this project is configured to se
Comment #10 on issue 3915 by waldron@gmail.com: Implement
exponentiation operator
https://code.google.com/p/v8/issues/detail?id=3915
Just another update, SpiderMonkey has landed this feature
https://bugzilla.mozilla.org/show_bug.cgi?id=1135708 and the proposal has
been advanced to Sta
lgtm
https://codereview.chromium.org/1269583002/
--
--
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
On 2015/07/24 06:32:10, Hannes Payer wrote:
https://codereview.chromium.org/1252053003/diff/1/src/heap/spaces.cc
File src/heap/spaces.cc (right):
https://codereview.chromium.org/1252053003/diff/1/src/heap/spaces.cc#newcode1478
src/heap/spaces.cc:1478: int aligned_size_in_bytes = size_in_bytes
On 2015/07/27 05:02:56, paul.l... wrote:
Cool stuff, Ross! Here's a first-cut MIPS port for
https://codereview.chromium.org/1245133002/
I still need to do the MIPS64, and take one more look at this before you
include
it in your CL. I'll do that in the morning my time. Any feedback welcome!
Updated with support for other architectures and a couple of fixes exposed
by
testing. PTAL.
On 2015/07/29 04:37:51, paul.l... wrote:
DBC: looks like your new use of r5 conflicts with r5 use in CEntryStub.
More
details in https://codereview.chromium.org/1253273002/.
This shouldn't be an is
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1265603002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1265603002/1
https://codereview.chromium.org/1265603002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
LGTM, thanks.
https://codereview.chromium.org/1265603002/
--
--
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 emai
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1254293006/40001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1254293006/40001
https://codereview.chromium.org/1254293006/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
lgtm.
https://codereview.chromium.org/1265603002/
--
--
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
https://codereview.chromium.org/1254293006/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):
https://codereview.chromium.org/1254293006/diff/1/src/flag-definitions.h#newcode283
src/flag-definitions.h:283: DEFINE_BOOL(ignition, true, "use ignition
interpreter")
On 2015/07/29 08:1
Updates:
Owner: ad...@chromium.org
Cc: yang...@chromium.org
Labels: Debugger
Comment #6 on issue 3659 by ad...@chromium.org: Classes: Make sure function
name inferrer works
https://code.google.com/p/v8/issues/detail?id=3659
(No comment was entered for this change.)
--
Updates:
Status: Duplicate
Mergedinto: 3659
Comment #3 on issue 4333 by ad...@chromium.org: Function name inferrer does
not work for ES6 classes
https://code.google.com/p/v8/issues/detail?id=4333
(No comment was entered for this change.)
--
You received this message because t
Updates:
Cc: rossb...@chromium.org little...@chromium.org
Comment #5 on issue 3659 by ad...@chromium.org: Classes: Make sure function
name inferrer works
https://code.google.com/p/v8/issues/detail?id=3659
Issue 4333 has been merged into this issue.
--
You received this message becaus
Try jobs failed on following builders:
v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel/builds/3179)
https://codereview.chromium.org/1254293006/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1254293006/20001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1254293006/20001
https://codereview.chromium.org/1254293006/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
https://codereview.chromium.org/1269583002/diff/1/src/heap/gc-idle-time-handler.cc
File src/heap/gc-idle-time-handler.cc (right):
https://codereview.chromium.org/1269583002/diff/1/src/heap/gc-idle-time-handler.cc#newcode205
src/heap/gc-idle-time-handler.cc:205:
idle_times_which_made_no_progress_
Hi all,
For the last month CQ posts an informative warning message if the author
has not signed CLA, and we're glad to see that this message greatly reduced
the percentage of authors w/o CLA. So now we've fully enabled SignCLA
verifier in CQ to make the check a real blocker.
Please let us know if
https://codereview.chromium.org/1269583002/diff/1/src/heap/gc-idle-time-handler.cc
File src/heap/gc-idle-time-handler.cc (right):
https://codereview.chromium.org/1269583002/diff/1/src/heap/gc-idle-time-handler.cc#newcode205
src/heap/gc-idle-time-handler.cc:205:
idle_times_which_made_no_progress_
Reviewers: Hannes Payer,
Message:
ptal
Description:
Fix idle notification for background tab.
The idle time handler should never return DONE or DO_SCAVENGE for
background tabs. Upon receiving DONE chrome will stop sending idle
notifications.
BUG=chromium:515174
LOG=NO
Please review this at ht
Comment #7 on issue 4334 by verwa...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
So that matches what I described in #2. Sounds like a good plan.
--
You received this message because this project is config
Comment #6 on issue 4334 by hara...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
Chatted with Jakob offline.
It seems that the problem is that window.innerWidth is still using a
data-type property. Once w
Comment #3 on issue 4245 by ad...@chromium.org: String.prototype.split
should use ToLength
https://code.google.com/p/v8/issues/detail?id=4245
This was discussed at the TC39 meeting yesterday, and Allen asked me to
file a spec bug: https://bugs.ecmascript.org/show_bug.cgi?id=4432
I think I
Comment #5 on issue 4334 by verwa...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
Yes, that's exactly what I want as well. In this particular case it's
likely because the innerWidth native setter does the
Comment #4 on issue 4334 by hara...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
The spec is here: https://heycam.github.io/webidl/#es-attributes
6. If the attribute is declared with a [Replaceable] extende
Comment #3 on issue 4334 by hara...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
I wasn't expecting that I can get a reply from you :)
We want to make the following change.
(Current behavior)
window.innerW
Comment #2 on issue 4334 by verwa...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
API accessors entirely behave like data properties, with the data
management handled by the accessor. If we don't call the
Reviewers: ulan, Benedikt Meurer,
Description:
[turbofan] Merge dependent Word32Equal on ARM64
Improve code generated for flag materialization.
BUG=
Please review this at https://codereview.chromium.org/1260733003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (
LGTM.
Thankfully the STATIC_ASSERT on the next line says that this change is
safe :-)
https://codereview.chromium.org/1265663002/
--
--
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 Gr
Updates:
Status: Available
Cc: jkumme...@chromium.org rossb...@chromium.org yang...@chromium.org
Labels: Type-FeatureRequest Area-Runtime Priority-Medium
Comment #1 on issue 4332 by habl...@chromium.org: JSON.parse() fails
without helpful line number of line offset infor
Updates:
Cc: yukishi...@chromium.org
Comment #1 on issue 4334 by hara...@chromium.org: API: Should
CreateDataProperty overwrite accessors, or call them?
https://code.google.com/p/v8/issues/detail?id=4334
(No comment was entered for this change.)
--
You received this message because t
Status: Available
Owner:
CC: hara...@chromium.org, verwa...@chromium.org, joc...@chromium.org,
jkumme...@chromium.org, rossb...@chromium.org
Labels: Type-Bug Priority-Medium Area-API
New issue 4334 by jkumme...@chromium.org: API: Should CreateDataProperty
overwrite accessors, or cal
Reviewers: Benedikt Meurer, danno,
Description:
Factor C call descriptor building into compiler/c-linkage.cc with inline
platform-specifics.
This is the first step in cutting the Gordian linkage/linkage-impl knot.
This basically changes the axis along which we organize call descriptor
building
On 2015/07/29 14:51:46, commit-bot: I haz the power wrote:
Dry run: CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1262833002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1262833002/1
Lgtm
https://codereview.chromium.
Reviewers: Jakob,
Description:
Assign more bits to safepoint table offset.
1 << 24 is apparently not large enough for some optimized code.
R=jkumme...@chromium.org
BUG=v8:4272
LOG=N
Please review this at https://codereview.chromium.org/1265663002/
Base URL: https://chromium.googlesource.com/v
Reviewers: Hannes Payer,
Description:
Reduce allowance in the first code page at start up.
This has become possible since we moved a few experimental
Javascript features to default.
R=hpa...@chromium.org
BUG=chromium:506044
LOG=N
Please review this at https://codereview.chromium.org/1262833002
Comment #3 on issue 4272 by vladisla...@gmail.com: SIGILL when running
Emscripten compiled script with --always_opt --nodead_code_elimination
--always_osr
https://code.google.com/p/v8/issues/detail?id=4272
The flags also make sense: the function is not hot enough to be optimized
otherwis
Mostly looking good, but I wonder about the cost of the IS_SIMD_VALUE
predicate.
https://codereview.chromium.org/1250733005/diff/210001/src/harmony-simd.js
File src/harmony-simd.js (right):
https://codereview.chromium.org/1250733005/diff/210001/src/harmony-simd.js#newcode35
src/harmony-simd.j
Comment #2 on issue 4272 by vladisla...@gmail.com: SIGILL when running
Emscripten compiled script with --always_opt --nodead_code_elimination
--always_osr
https://code.google.com/p/v8/issues/detail?id=4272
It seems, the generated code for the function is simply too large. This
causes saf
Reviewers: Michael Starzinger, rossberg,
Message:
PTAL
Description:
Move final parts of class literal setup into a single runtime call
This avoids multiple ToFastProperties runtime calls and additional stack
pushes in strong mode.
BUG=
Please review this at https://codereview.chromium.org/126
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1256323004/20001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1256323004/20001
https://codereview.chromium.org/1256323004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1259723002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1259723002/1
https://codereview.chromium.org/1259723002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
Hi Benedikt,
Here is a fix for the bug we saw the other day, where an IC slot is
foolishly
allocated for a VariableProxy on the LHS of an assignment operation. (also
Property).
Thanks for the look,
--Michael
https://codereview.chromium.org/1262803002/
--
--
v8-dev mailing list
v8-dev@googlegr
Reviewers: ,
Message:
Hi Benedikt,
Here is a fix for the bug we saw the other day, where an IC slot is
foolishly
allocated for a VariableProxy on the LHS of an assignment operation. (also
Property).
Thanks for the look,
--Michael
Description:
Stop overallocating feedback vector slots.
When a
Reviewers: Michael Starzinger,
Message:
PTAL
Description:
Tail call from TurboFan into select runtime calls
Please review this at https://codereview.chromium.org/1254073003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+33, -0 lines):
M src/compiler/js-contex
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1258743004/20001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1258743004/20001
https://codereview.chromium.org/1258743004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
lgtm
https://codereview.chromium.org/1256323004/diff/20001/src/runtime.js
File src/runtime.js (left):
https://codereview.chromium.org/1256323004/diff/20001/src/runtime.js#oldcode769
src/runtime.js:769: if (IS_SYMBOL_WRAPPER(x)) throw
MakeTypeError(kSymbolToPrimitive);
On 2015/07/28 19:07:30,
Patchset 1 (id:??) landed as
https://crrev.com/2c16d81b256ed664c376342313558a37bbce0131
Cr-Commit-Position: refs/heads/master@{#29905}
https://codereview.chromium.org/1260423002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this messag
Committed patchset #1 (id:1)
https://codereview.chromium.org/1260423002/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1260423002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1260423002/1
https://codereview.chromium.org/1260423002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
https://codereview.chromium.org/1268463002/ should fix those failures,
let's try
CQing again...
https://codereview.chromium.org/1260423002/
--
--
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 G
Patchset 1 (id:??) landed as
https://crrev.com/485aca6df6065790c7454348eb3aac51b6e24616
Cr-Commit-Position: refs/heads/master@{#29904}
https://codereview.chromium.org/1268463002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this messag
Committed patchset #1 (id:1)
https://codereview.chromium.org/1268463002/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1268463002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1268463002/1
https://codereview.chromium.org/1268463002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
Reviewers: paul.l..., balazs.kilvady, gergely.kis.imgtec,
Message:
PTAL.
Description:
MIPS64: Fix the integer division in crankshaft.
Replaces the 64-bit div instruction with 32-bit division in DivI.
Also fixes the Ddiv implementation in the simulator.
TEST=mjsunit/asm/int32div
BUG=
Please re
lgtm
https://codereview.chromium.org/1262113002/
--
--
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
LGTM.
https://codereview.chromium.org/1263723002/
--
--
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
lgtm
https://codereview.chromium.org/1268463002/
--
--
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
Updates:
Owner: ad...@chromium.org
Cc: -ad...@chromium.org rossb...@chromium.org
Comment #2 on issue 4333 by rossb...@chromium.org: Function name inferrer
does not work for ES6 classes
https://code.google.com/p/v8/issues/detail?id=4333
(No comment was entered for this change.)
Reviewers: Jakob,
Description:
Debugger: skip function prologue when computing redirect PC.
R=jkumme...@chromium.org
Please review this at https://codereview.chromium.org/1268463002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+18, -5 lines):
M src/debug.cc
Committed patchset #2 (id:20001) manually as
aa84551622799c6c44b8ee60ea6c40405465177a (tree was closed).
https://codereview.chromium.org/1263773002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed t
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1264603002/20001
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1264603002/20001
https://codereview.chromium.org/1264603002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
ht
Try jobs failed on following builders:
v8_linux64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel/builds/8156)
https://codereview.chromium.org/1260423002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
-
lgtm
https://codereview.chromium.org/1264603002/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1264603002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1264603002/1
https://codereview.chromium.org/1264603002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
Comment #1 on issue 4333 by yang...@chromium.org: Function name inferrer
does not work for ES6 classes
https://code.google.com/p/v8/issues/detail?id=4333
Here's the test case: https://codereview.chromium.org/1264603002
--
You received this message because this project is configured to send a
Reviewers: mvstanton,
Description:
Add test for referring function name for classes.
R=mvstan...@chromium.org
BUG=v8:4333
LOG=N
Please review this at https://codereview.chromium.org/1264603002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+44, -0 lines):
M te
Status: Assigned
Owner: rossb...@chromium.org
CC: ad...@chromium.org, little...@chromium.org
Labels: Type-Bug Priority-Medium Harmony Debugger
New issue 4333 by yang...@chromium.org: Function name inferrer does not
work for ES6 classes
https://code.google.com/p/v8/issues/detail?id=4333
We us
On 2015/07/29 09:26:49, Jakob wrote:
Created Revert of Reland^2 "Enable loads and stores to global vars through
property cell shortcuts installed into par...
lgtm
https://codereview.chromium.org/1260423002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-de
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1260423002/1
View timeline at
https://chromium-cq-status.appspot.com/patch-timeline/1260423002/1
https://codereview.chromium.org/1260423002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://gro
Comment #3 on issue 1801 by mariusz.pala: V8/Chrome for Windows Only does
not respect United States DST rules prior to 2007
https://code.google.com/p/v8/issues/detail?id=1801
Note that it works as intended in IE10 and IE11 and in Chrome on OSX. But
it doesn't work correctly in Chrome on Win
Reviewers: Igor Sheludko, v8-mips-ports_googlegroups.com, Paul Lind,
Benedikt Meurer,
Message:
Created Revert of Reland^2 "Enable loads and stores to global vars through
property cell shortcuts installed into par...
Description:
Revert of Reland^2 "Enable loads and stores to global vars throug
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1260423002/ by jkumme...@chromium.org.
The reason for reverting is: Suspected to cause Canary crashes.
https://codereview.chromium.org/1254723004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://
Comment #2 on issue 1801 by mariusz.pala: V8/Chrome for Windows Only does
not respect United States DST rules prior to 2007
https://code.google.com/p/v8/issues/detail?id=1801
Hi, this is a really old bug, can you finally fix it please? Thank you
--
You received this message because this proj
Right on, LGTM.
https://codereview.chromium.org/1263773002/
--
--
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 e
Thanks Michael, that makes it simpler! Done.
https://codereview.chromium.org/1263773002/
--
--
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 thi
Nice. Use site_context->top() and you won't have to pass the new
parameter...
https://codereview.chromium.org/1263773002/
--
--
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-d
Reviewers: mvstanton, jochen,
Description:
Pretenuring decision of outermost literal is propagated to inner literals.
Literals should stay together to simplify allocation folding. This CL takes
the
pretenuring information from the outermost literal and applies it to the
inner
liberals.
BUG
Reviewers: Benedikt Meurer, titzer,
Message:
Not for landing yet, but I'd appreciate feedback before doing the rest of
the
platform ports.
Description:
[turbofan]: Implement tail calls with differing stack parameter counts
WORK IN PROGRESS, I'D APPRECIATE FEEDBACK BEFORE DOING THE OTHER PLAT
On 2015/07/29 08:33:54, mvstanton wrote:
Hi Yang, here is a fix for the function name inferrer.
Recent changes in scoping caused functions that were lazily parsed to be
eagerly
parsed. It appears that the eager parse case didn't even have the
possibility
for the function name inferrer to w
1 - 100 of 117 matches
Mail list logo