[v8-dev] Re: Reduce queue size in dead code elimination by eagerly processing live instructions. (issue 27178002)

2013-10-14 Thread titzer
https://codereview.chromium.org/27178002/ -- -- 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: Reduce queue size in dead code elimination by eagerly processing live instructions. (issue 27178002)

2013-10-14 Thread rmcilroy
lgtm with one nit. Nice, thanks! https://codereview.chromium.org/27178002/diff/1/src/hydrogen-dce.cc File src/hydrogen-dce.cc (right): https://codereview.chromium.org/27178002/diff/1/src/hydrogen-dce.cc#newcode35 src/hydrogen-dce.cc:35: HValue* instr, ZoneListHValue** worklist) { add an

[v8-dev] Re: Reduce queue size in dead code elimination by eagerly processing live instructions. (issue 27178002)

2013-10-14 Thread titzer
On 2013/10/14 12:43:29, rmcilroy wrote: lgtm with one nit. Nice, thanks! https://codereview.chromium.org/27178002/diff/1/src/hydrogen-dce.cc File src/hydrogen-dce.cc (right): https://codereview.chromium.org/27178002/diff/1/src/hydrogen-dce.cc#newcode35 src/hydrogen-dce.cc:35: HValue*

[v8-dev] Re: Reduce queue size in dead code elimination by eagerly processing live instructions. (issue 27178002)

2013-10-14 Thread titzer
Committed patchset #2 manually as r17196 (presubmit successful). https://codereview.chromium.org/27178002/ -- -- 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