[v8-dev] Re: [turbofan] Clarify comment about Parameter indexing. (issue 1329043002 by mstarzin...@chromium.org)

2015-09-07 Thread rmcilroy
Ahh yes, it was the variable indexes which had the '-1', that explains my confusion. Thanks! lgtm. https://codereview.chromium.org/1329043002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to

[v8-dev] Re: [turbofan] Clarify comment about Parameter indexing. (issue 1329043002 by mstarzin...@chromium.org)

2015-09-07 Thread oth
On 2015/09/07 08:58:38, Michael Starzinger wrote: Thanks Michael. LGTM. https://codereview.chromium.org/1329043002/ -- -- 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"

[v8-dev] Re: [turbofan] Clarify comment about Parameter indexing. (issue 1329043002 by mstarzin...@chromium.org)

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

[v8-dev] Re: [turbofan] Clarify comment about Parameter indexing. (issue 1329043002 by mstarzin...@chromium.org)

2015-09-07 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #1 (id:1) https://codereview.chromium.org/1329043002/ -- -- 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] Re: [turbofan] Clarify comment about Parameter indexing. (issue 1329043002 by mstarzin...@chromium.org)

2015-09-07 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1329043002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1329043002/1 https://codereview.chromium.org/1329043002/ -- -- v8-dev mailing list v8-dev@googlegroups.com