lgtm
https://codereview.chromium.org/515793002/
--
--
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
PTAL
https://codereview.chromium.org/512123002/
--
--
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 it,
Revision: 23472
Author: machenb...@chromium.org
Date: Thu Aug 28 06:56:00 2014 UTC
Log: Revert Remove C++11-ism, until all bots support it.
This reverts commit r23463 as by now shouldn't be a syntax error anymore
in chromium.
TBR=vogelh...@chromium.org, svenpa...@chromium.org
Committed patchset #1 (id:1) manually as 23472 (presubmit successful).
https://codereview.chromium.org/512123002/
--
--
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.
Committed patchset #1 (id:1) manually as 23473 (presubmit successful).
https://codereview.chromium.org/53002/
--
--
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.
PTAL
https://codereview.chromium.org/53002/
--
--
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 it,
Revision: 23473
Author: machenb...@chromium.org
Date: Thu Aug 28 07:01:56 2014 UTC
Log: Reland Enable C++11 tests on windows.
TBR=svenpa...@chromium.org
BUG=400033
LOG=n
Review URL: https://codereview.chromium.org/53002
https://code.google.com/p/v8/source/detail?r=23473
Committed patchset #1 manually as 23474 (presubmit successful).
https://codereview.chromium.org/513653003/
--
--
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
Revision: 23474
Author: yang...@chromium.org
Date: Thu Aug 28 07:02:53 2014 UTC
Log: Sub-minor-key-ify four HydrogenCodeStubs.
- FastNewContextStub
- FastCloneShallowArrayStub
- ToBooleanStub
- ElementsTransitionAndStoreStub.
R=mvstan...@chromium.org
Review URL:
Revision: 23475
Author: machenb...@chromium.org
Date: Thu Aug 28 07:03:22 2014 UTC
Log: Version 3.29.24 (based on bleeding_edge revision r23472)
Tweaks to generate XP-compatible .exes (Chromium issue 407517).
Performance and stability improvements on all platforms.
Revision: 23476
Author: machenb...@chromium.org
Date: Thu Aug 28 07:04:13 2014 UTC
Log: Tagging version 3.29.24
https://code.google.com/p/v8/source/detail?r=23476
Added:
/tags/3.29.24
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You
Committed patchset #1 manually as 23477 (presubmit successful).
https://codereview.chromium.org/511853002/
--
--
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
Revision: 23477
Author: yang...@chromium.org
Date: Thu Aug 28 07:12:16 2014 UTC
Log: Sub-minor-key-ify seven HydrogenCodeStubs.
- FastCloneShallowObjectStub
- StringAddStub
- LoadFastElementStub
- StoreFastElementStub
- TransitionElementsKindStub
- ArrayConstructorStubBase
-
https://codereview.chromium.org/488003005/diff/40001/test/mjsunit/es6/math-hyperbolic.js
File test/mjsunit/es6/math-hyperbolic.js (right):
https://codereview.chromium.org/488003005/diff/40001/test/mjsunit/es6/math-hyperbolic.js#newcode162
test/mjsunit/es6/math-hyperbolic.js:162:
Revision: 23478
Author: svenpa...@chromium.org
Date: Thu Aug 28 07:30:58 2014 UTC
Log: Fix disassembly redirection from stdout into a file.
Pass \n, \r and \t through OStream without escaping.
BUG=
R=svenpa...@chromium.org
Review URL: https://codereview.chromium.org/458533002
Patch
Committed patchset #2 (id:20001) manually as 23478 (presubmit successful).
https://codereview.chromium.org/458533002/
--
--
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
On 2014/08/27 17:54:51, Michael Achenbach wrote:
On 2014/08/26 10:49:38, rmcilroy wrote:
On 2014/08/23 18:38:48, Michael Achenbach wrote:
Committed patchset #1 manually as 23325 (tree was closed).
Thanks for reverting (apologies for not spotting this, the bot took a
while
to
go red and
Committed patchset #11 manually as 23479 (presubmit successful).
https://codereview.chromium.org/473263004/
--
--
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
Revision: 23479
Author: sigu...@chromium.org
Date: Thu Aug 28 08:39:24 2014 UTC
Log: Remove dependency from generic lowering on compilation info for
determining strictness and builtins.
This makes the graphs compositional for inlining (i.e. we can now inline a
strict function
Revision: 23480
Author: sigu...@chromium.org
Date: Thu Aug 28 08:47:11 2014 UTC
Log: Fix build.
GCC needs definitions for constant members (clang doesn't).
TBR=mstarzin...@chromium.org
Review URL: https://codereview.chromium.org/511133002
Reviewers: Michael Starzinger,
Message:
Committed patchset #1 manually as 23480 (tree was closed).
Description:
Fix build.
GCC needs definitions for constant members (clang doesn't).
TBR=mstarzin...@chromium.org
Committed: https://code.google.com/p/v8/source/detail?r=23480
Please review
https://codereview.chromium.org/511133002/diff/1/test/cctest/compiler/function-tester.h
File test/cctest/compiler/function-tester.h (left):
https://codereview.chromium.org/511133002/diff/1/test/cctest/compiler/function-tester.h#oldcode28
test/cctest/compiler/function-tester.h:28: const uint32_t
Reviewers: Sven Panne, vogelheim,
Message:
Committed patchset #1 (id:1) manually as 23481 (presubmit successful).
Description:
Revert Revert Remove C++11-ism, until all bots support it.
This reverts commit 1f07f57a4a13985f3a48b06d2848dd3bff165b4f.
AOSP bot is still not ready.
Revision: 23481
Author: machenb...@chromium.org
Date: Thu Aug 28 09:20:49 2014 UTC
Log: Revert Revert Remove C++11-ism, until all bots support it.
This reverts commit 1f07f57a4a13985f3a48b06d2848dd3bff165b4f.
AOSP bot is still not ready.
Revision: 23482
Author: machenb...@chromium.org
Date: Thu Aug 28 09:24:16 2014 UTC
Log: Version 3.29.25 (based on bleeding_edge revision r23481)
Performance and stability improvements on all platforms.
https://code.google.com/p/v8/source/detail?r=23482
Modified:
/trunk/ChangeLog
Reviewers: Michael Starzinger,
Message:
PTAL
Description:
Add more inlining unit tests.
- Scheduler can inline a function with a loop.
- Inlining strict into non-strict works correctly.
- Inlining non-strict into strict works correctly.
R=mstarzin...@chromium.org
Please review this at
https://codereview.chromium.org/511133002/diff/1/test/cctest/compiler/function-tester.h
File test/cctest/compiler/function-tester.h (left):
https://codereview.chromium.org/511133002/diff/1/test/cctest/compiler/function-tester.h#oldcode28
test/cctest/compiler/function-tester.h:28: const uint32_t
Revision: 23483
Author: machenb...@chromium.org
Date: Thu Aug 28 09:24:46 2014 UTC
Log: Tagging version 3.29.25
https://code.google.com/p/v8/source/detail?r=23483
Added:
/tags/3.29.25
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You
hello. Jarin.
Please help to commit it.
thanks
https://codereview.chromium.org/515793002/
--
--
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
On 2014/08/28 08:30:11, ulan wrote:
On 2014/08/27 17:54:51, Michael Achenbach wrote:
On 2014/08/26 10:49:38, rmcilroy wrote:
On 2014/08/23 18:38:48, Michael Achenbach wrote:
Committed patchset #1 manually as 23325 (tree was closed).
Thanks for reverting (apologies for not spotting
Comment #12 on issue 1942 by waw...@gmail.com: JavaScript Harmony freezes
Google Maps
https://code.google.com/p/v8/issues/detail?id=1942
I was tracking down a bug today and found this thread while searching for
similar error messages.
The error message was fairly strange to me:
Reviewers: titzer,
Description:
Preliminary lowering of typed array loads in TF.
R=tit...@chromium.org
TEST=cctest/test-run-properties/TypedArrayLoad
Please review this at https://codereview.chromium.org/516853002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files
https://codereview.chromium.org/516853002/diff/1/src/compiler/js-typed-lowering.cc
File src/compiler/js-typed-lowering.cc (right):
https://codereview.chromium.org/516853002/diff/1/src/compiler/js-typed-lowering.cc#newcode502
src/compiler/js-typed-lowering.cc:502: FieldAccess
PTAL. Could a v8 team member review?
https://codereview.chromium.org/492303004/
--
--
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
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
Why is it necessary for MIPS? Would the same also apply for ARM?
https://codereview.chromium.org/492303004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You
https://codereview.chromium.org/515103002/diff/1/test/cctest/compiler/test-run-inlining.cc
File test/cctest/compiler/test-run-inlining.cc (right):
https://codereview.chromium.org/515103002/diff/1/test/cctest/compiler/test-run-inlining.cc#newcode234
test/cctest/compiler/test-run-inlining.cc:234:
On 2014/08/27 17:47:13, Slava Chigrin wrote:
https://codereview.chromium.org/514783002/diff/1/src/serialize.cc
File src/serialize.cc (right):
https://codereview.chromium.org/514783002/diff/1/src/serialize.cc#newcode1640
src/serialize.cc:1640: }
Seems, this increment was missed, so pointers go
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
Why is it necessary for MIPS? Would the same also apply for ARM?
Thank you for the quick review.
The same has done for arm/V8_OOL_CONSTANT_POOL arches in r r23242. Wee
CQ is trying da patch. Follow status at
https://v8-status.appspot.com/cq/vchig...@yandex-team.ru/514783002/1
https://codereview.chromium.org/514783002/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are
No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do
Oh! Sorry, I thought I landed it but somehow I didn't :( Thanks a bunch for
landing it for me.
// Vyacheslav Egorov
On Thu, Aug 28, 2014 at 9:31 AM, svenpa...@chromium.org wrote:
Committed patchset #2 (id:20001) manually as 23478 (presubmit successful).
On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See
On 2014/08/28 10:30:17, Yang wrote:
On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star
On 2014/08/28 09:54:43, balazs.kilvady wrote:
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
Why is it necessary for MIPS? Would the same also apply for ARM?
Thank you for the quick review.
The same has done for
Comments addressed. PTAL.
https://codereview.chromium.org/516853002/diff/1/src/compiler/js-typed-lowering.cc
File src/compiler/js-typed-lowering.cc (right):
https://codereview.chromium.org/516853002/diff/1/src/compiler/js-typed-lowering.cc#newcode502
src/compiler/js-typed-lowering.cc:502:
On 2014/08/28 10:38:09, Slava Chigrin wrote:
On 2014/08/28 10:30:17, Yang wrote:
On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a
On 2014/08/28 10:38:15, Yang wrote:
On 2014/08/28 09:54:43, balazs.kilvady wrote:
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
Why is it necessary for MIPS? Would the same also apply for ARM?
Thank you for the
On 2014/08/28 10:47:08, rmcilroy wrote:
On 2014/08/28 10:38:15, Yang wrote:
On 2014/08/28 09:54:43, balazs.kilvady wrote:
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
Why is it necessary for MIPS? Would the
Revision: 23484
Author: yang...@chromium.org
Date: Thu Aug 28 10:51:43 2014 UTC
Log: Fix serialization references to Cell objects.
R=yang...@chromium.org
Review URL: https://codereview.chromium.org/514783002
Patch from Slava Chigrin vchig...@yandex-team.ru.
Committed patchset #1 (id:1) manually as 23484 (presubmit successful).
https://codereview.chromium.org/514783002/
--
--
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.
PTAL
https://codereview.chromium.org/512183004/
--
--
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 it,
https://codereview.chromium.org/515103002/diff/1/test/cctest/compiler/test-run-inlining.cc
File test/cctest/compiler/test-run-inlining.cc (right):
https://codereview.chromium.org/515103002/diff/1/test/cctest/compiler/test-run-inlining.cc#newcode234
test/cctest/compiler/test-run-inlining.cc:234:
Committed patchset #2 manually as 23485 (presubmit successful).
https://codereview.chromium.org/515793002/
--
--
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
Revision: 23485
Author: ja...@chromium.org
Date: Thu Aug 28 11:06:26 2014 UTC
Log: Disable running some instruction selector test on unsupported
backends.
BUG=
R=ja...@chromium.org
Review URL: https://codereview.chromium.org/515793002
Patch from Chunyang Dai
Reviewers: Michael Starzinger,
Description:
Use TruncateFloat64ToInt32 instead of ChangeFloat64ToInt32 in lowering of
simplified operators NumberToInt32/NumberToUint32.
R=mstarzin...@chromium.org
BUG=
Please review this at https://codereview.chromium.org/509653003/
SVN Base:
lgtm
https://codereview.chromium.org/512183004/
--
--
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/516853002/diff/1/src/compiler/js-typed-lowering.cc
File src/compiler/js-typed-lowering.cc (right):
https://codereview.chromium.org/516853002/diff/1/src/compiler/js-typed-lowering.cc#newcode502
src/compiler/js-typed-lowering.cc:502: FieldAccess
Committed patchset #1 (id:1) manually as 23486 (presubmit successful).
https://codereview.chromium.org/512183004/
--
--
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.
Revision: 23486
Author: machenb...@chromium.org
Date: Thu Aug 28 11:09:46 2014 UTC
Log: Add android tools to v8's deps.
BUG=408510
LOG=n
R=hin...@google.com
Review URL: https://codereview.chromium.org/512183004
https://code.google.com/p/v8/source/detail?r=23486
Modified:
https://codereview.chromium.org/516853002/diff/20001/src/compiler/simplified-operator.h
File src/compiler/simplified-operator.h (right):
https://codereview.chromium.org/516853002/diff/20001/src/compiler/simplified-operator.h#newcode99
src/compiler/simplified-operator.h:99: class Access : public
Updates:
Summary: Ensure that Turbofan handles Harmony features gracefully
Status: Assigned
Owner: mstarzin...@chromium.org
Cc: rossb...@chromium.org
Labels: Type-Bug Harmony Turbofan
Comment #4 on issue 3527 by v8.team@gmail.com: Ensure that Turbofan
On 2014/08/28 10:50:17, Yang wrote:
On 2014/08/28 10:47:08, rmcilroy wrote:
On 2014/08/28 10:38:15, Yang wrote:
On 2014/08/28 09:54:43, balazs.kilvady wrote:
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady wrote:
PTAL. Could a v8 team member review?
On 2014/08/28 11:14:04, balazs.kilvady wrote:
On 2014/08/28 10:50:17, Yang wrote:
On 2014/08/28 10:47:08, rmcilroy wrote:
On 2014/08/28 10:38:15, Yang wrote:
On 2014/08/28 09:54:43, balazs.kilvady wrote:
On 2014/08/28 09:48:54, Yang wrote:
On 2014/08/28 09:45:19, balazs.kilvady
Comments addressed. PTAL.
https://codereview.chromium.org/516853002/diff/20001/src/compiler/simplified-operator.h
File src/compiler/simplified-operator.h (right):
https://codereview.chromium.org/516853002/diff/20001/src/compiler/simplified-operator.h#newcode99
Revision: 23487
Author: balazs.kilv...@imgtec.com
Date: Thu Aug 28 11:27:21 2014 UTC
Log: MIPS: Fix [de]serialize problem of root objects.
BUG
TEST=mjsunit/deserialize-reference
R=yang...@chromium.org
Review URL: https://codereview.chromium.org/492303004
Committed patchset #2 manually as r23487.
https://codereview.chromium.org/492303004/
--
--
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
LGTM.
https://codereview.chromium.org/515103002/
--
--
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 2014/08/27 17:06:01, Slava Chigrin wrote:
https://codereview.chromium.org/510013002/diff/1/src/serialize.cc
File src/serialize.cc (right):
https://codereview.chromium.org/510013002/diff/1/src/serialize.cc#newcode1545
src/serialize.cc:1545: }
Before that sometimes current[repeat_count] can
lgtm
https://codereview.chromium.org/516853002/
--
--
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:
Cc: tit...@chromium.org
Labels: -Turbofan TurboFan
Comment #5 on issue 3527 by mstarzin...@chromium.org: Ensure that Turbofan
handles Harmony features gracefully
https://code.google.com/p/v8/issues/detail?id=3527
Yep, the blacklist needs to be fixed before we
Reviewers: jarin,
Description:
Add MachineSignature, which is an encapsulation of the machine types for
parameters and return values in a graph.
This utility will be used to simplify Linkage and fix representation
inference
to work with graphs where parameters and return values are something
Revision: 23488
Author: yang...@chromium.org
Date: Thu Aug 28 11:45:20 2014 UTC
Log: Fix rare access violation during JS heap serialization.
R=yang...@chromium.org
Review URL: https://codereview.chromium.org/510013002
Patch from Slava Chigrin vchig...@yandex-team.ru.
Committed patchset #1 (id:1) manually as 23488 (presubmit successful).
https://codereview.chromium.org/510013002/
--
--
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.
https://codereview.chromium.org/514643002/diff/1/src/compiler/js-generic-lowering.cc
File src/compiler/js-generic-lowering.cc (right):
https://codereview.chromium.org/514643002/diff/1/src/compiler/js-generic-lowering.cc#newcode224
src/compiler/js-generic-lowering.cc:224:
On 2014/08/28 11:45:29, Yang wrote:
Committed patchset #1 (id:1) manually as 23488 (presubmit successful).
Thank you so much for help! I have one little question about serializer:
why it
have so big macro CASE_BODY in Deserializer::ReadChunk? May be using inline
templatized function
On 2014/08/28 11:54:03, Slava Chigrin wrote:
On 2014/08/28 11:45:29, Yang wrote:
Committed patchset #1 (id:1) manually as 23488 (presubmit successful).
Thank you so much for help! I have one little question about serializer:
why
it
have so big macro CASE_BODY in Deserializer::ReadChunk?
lgtm
https://codereview.chromium.org/515173002/
--
--
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
Revision: 23489
Author: sigu...@chromium.org
Date: Thu Aug 28 12:18:25 2014 UTC
Log: Simplify inlining now that the scheduler is smart(er).
* Only control adjustment is to move everything from the inlinee's
start block to the block the call was in.
* Add a unit test to ensure that
Committed patchset #1 manually as 23489 (presubmit successful).
https://codereview.chromium.org/508953002/
--
--
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
Reviewers: Michael Starzinger, titzer,
Description:
Better typing and type verification
- Extend typer to handle Simplified nodes.
- Extend verifier to check types of JS and Simplified nodes.
- Untyped nodes now contain NULL as types, enforcing hard failure.
- Distinguish between typed and
LGTM.
https://codereview.chromium.org/509653003/
--
--
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/509343002/diff/1/src/compiler/generic-node-inl.h
File src/compiler/generic-node-inl.h (right):
https://codereview.chromium.org/509343002/diff/1/src/compiler/generic-node-inl.h#newcode26
src/compiler/generic-node-inl.h:26: inputs_.static_ =
Committed patchset #1 manually as 23490 (presubmit successful).
https://codereview.chromium.org/515173002/
--
--
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
Revision: 23490
Author: tit...@chromium.org
Date: Thu Aug 28 13:17:38 2014 UTC
Log: Add MachineSignature, which is an encapsulation of the machine
types for parameters and return values in a graph.
This utility will be used to simplify Linkage and fix representation
inference
to
Committed patchset #1 manually as 23491 (presubmit successful).
https://codereview.chromium.org/509653003/
--
--
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
Revision: 23491
Author: tit...@chromium.org
Date: Thu Aug 28 13:31:55 2014 UTC
Log: Use TruncateFloat64ToInt32 instead of ChangeFloat64ToInt32 in
lowering of simplified operators NumberToInt32/NumberToUint32.
R=mstarzin...@chromium.org
BUG=
Review URL:
Reviewers: Toon Verwaest,
Description:
Do not expose termination exceptions to the API.
R=verwa...@chromium.org
BUG=403509
Please review this at https://codereview.chromium.org/516913003/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+21, -0 lines):
M
lgtm
https://codereview.chromium.org/496183003/
--
--
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
There are still some obscure linker errors for 64bit div/mod operations, but
please PTAL nevertheless...
https://codereview.chromium.org/513923005/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed
On 2014/08/28 13:49:02, Sven Panne wrote:
There are still some obscure linker errors for 64bit div/mod operations,
but
please PTAL nevertheless...
... and sem_timedwait still has no replacement. This will come soon, just
want
to get some early feedback.
Status: New
Owner:
New issue 3543 by n42...@gmail.com: Install of V8 on REHL 6 not connected
to the internet.
https://code.google.com/p/v8/issues/detail?id=3543
I am attempting to install V8 on a Linux server that is not connected to
the internet and can not be connected to the
Committed patchset #5 (id:80001) manually as 23492 (presubmit successful).
https://codereview.chromium.org/516853002/
--
--
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
Revision: 23492
Author: mstarzin...@chromium.org
Date: Thu Aug 28 14:35:11 2014 UTC
Log: Preliminary lowering of typed array loads in TF.
R=tit...@chromium.org
TEST=cctest/test-run-properties/TypedArrayLoad
Review URL: https://codereview.chromium.org/516853002
Committed patchset #2 (id:20001) manually as 23493 (presubmit successful).
https://codereview.chromium.org/496183003/
--
--
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
Revision: 23493
Author: machenb...@chromium.org
Date: Thu Aug 28 14:42:24 2014 UTC
Log: Let benchmark runner add summary traces.
BUG=374740
LOG=n
R=ja...@chromium.org
Review URL: https://codereview.chromium.org/496183003
https://code.google.com/p/v8/source/detail?r=23493
Modified:
Reviewers: titzer,
Description:
Move constructors for [Element/Field]Access into AccessBuilder.
R=tit...@chromium.org
TEST=cctest/test-simplified-lowering
Please review this at https://codereview.chromium.org/516943002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected
Could you take a look, please?
https://codereview.chromium.org/515723004/
--
--
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
Revision: 23494
Author: mstarzin...@chromium.org
Date: Thu Aug 28 15:19:05 2014 UTC
Log: Skip Windows failure after r23492.
TBR=vogelh...@chromium.org
TEST=cctest/test-run-properties/TypedArrayLoad
Review URL: https://codereview.chromium.org/513223002
Committed patchset #2 (id:20001) manually as 23494 (tree was closed).
https://codereview.chromium.org/513223002/
--
--
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.
Reviewers: Michael Achenbach,
Description:
[Auto-roll] Bump up version to 3.29.26.0
TBR=machenb...@chromium.org
Please review this at https://codereview.chromium.org/516973002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -1 lines):
M src/version.cc
1 - 100 of 140 matches
Mail list logo