Fix missing visitation of effect inputs to loads and stores.

Fix missing visitation of effect inputs to loads and stores.

R=titzer@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=23641

[titzer, 2014-09-03 09:03:57]

https://codereview.chromium.org/533773002/diff/1/src/compiler/simplified-lowe...
File src/compiler/simplified-lowering.cc (right):

https://codereview.chromium.org/533773002/diff/1/src/compiler/simplified-lowe...
src/compiler/simplified-lowering.cc:191: void ProcessEffect(Node* node) {
Can we make this ProcessRemainingInputs(Node*, int index) and just iterate from
the index to the end?

[Michael Starzinger, 2014-09-03 10:49:59]

Addressed comments. PTAL.

https://codereview.chromium.org/533773002/diff/1/src/compiler/simplified-lowe...
File src/compiler/simplified-lowering.cc (right):

https://codereview.chromium.org/533773002/diff/1/src/compiler/simplified-lowe...
src/compiler/simplified-lowering.cc:191: void ProcessEffect(Node* node) {
On 2014/09/03 09:03:57, titzer wrote:
> Can we make this ProcessRemainingInputs(Node*, int index) and just iterate
from
> the index to the end?

Done.

[Michael Starzinger, 2014-09-03 10:52:10]

Patchset #2 (id:20001) has been deleted

[titzer, 2014-09-03 11:14:26]

lgtm

[Michael Starzinger, 2014-09-03 11:30:14]

Committed patchset #2 (id:40001) manually as 23641 (presubmit successful).