[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread titzer
lgtm https://codereview.chromium.org/1289863003/diff/60001/src/compiler/interpreter-assembler.h File src/compiler/interpreter-assembler.h (right): https://codereview.chromium.org/1289863003/diff/60001/src/compiler/interpreter-assembler.h#newcode96 src/compiler/interpreter-assembler.h:96:

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread rmcilroy
Ping? https://codereview.chromium.org/1289863003/ -- -- 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: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1289863003/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1289863003/80001 https://codereview.chromium.org/1289863003/ -- -- v8-dev mailing list v8-dev@googlegroups.com

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread rmcilroy
https://codereview.chromium.org/1289863003/ -- -- 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, send

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread rmcilroy
https://codereview.chromium.org/1289863003/diff/60001/src/compiler/interpreter-assembler.h File src/compiler/interpreter-assembler.h (right): https://codereview.chromium.org/1289863003/diff/60001/src/compiler/interpreter-assembler.h#newcode96 src/compiler/interpreter-assembler.h:96: Node*

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 4 (id:??) landed as https://crrev.com/00df60d1c6943a10fb5ca84fce2c017dcd2001f5 Cr-Commit-Position: refs/heads/master@{#30219} https://codereview.chromium.org/1289863003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-18 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #4 (id:80001) https://codereview.chromium.org/1289863003/ -- -- 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: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-14 Thread rmcilroy
Thanks for the review. Comments addressed, PTAL. https://codereview.chromium.org/1289863003/diff/20001/src/compiler/interpreter-assembler.h File src/compiler/interpreter-assembler.h (right): https://codereview.chromium.org/1289863003/diff/20001/src/compiler/interpreter-assembler.h#newcode40

[v8-dev] Re: [interpreter]: Changes to interpreter builtins for accumulator and register file registers. (issue 1289863003 by rmcil...@chromium.org)

2015-08-14 Thread titzer
https://codereview.chromium.org/1289863003/diff/20001/src/compiler/interpreter-assembler.h File src/compiler/interpreter-assembler.h (right): https://codereview.chromium.org/1289863003/diff/20001/src/compiler/interpreter-assembler.h#newcode40 src/compiler/interpreter-assembler.h:40: Node*