[fullcodegen] Factor out VisitProperty from architectures.

[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org

Committed: https://crrev.com/d0e84c8ebd43b3da41e4154f349f42de60448cb2
Cr-Commit-Position: refs/heads/master@{#36111}

[Michael Starzinger, 2016-05-09 14:13:03]

Description was changed from

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=
==========

to

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org
==========

[Michael Starzinger, 2016-05-09 14:13:03]

mstarzinger@chromium.org changed reviewers:
+ mvstanton@chromium.org

[titzer, 2016-05-09 14:15:32]

titzer@chromium.org changed reviewers:
+ titzer@chromium.org

[titzer, 2016-05-09 14:15:33]

lgtm

https://codereview.chromium.org/1962663002/diff/1/src/full-codegen/full-codeg...
File src/full-codegen/full-codegen.cc (right):

https://codereview.chromium.org/1962663002/diff/1/src/full-codegen/full-codeg...
src/full-codegen/full-codegen.cc:823: void
FullCodeGenerator::VisitProperty(Property* expr) {
Why would do this? :)

[mvstanton, 2016-05-09 14:17:26]

lgtm

[Michael Starzinger, 2016-05-09 14:29:21]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-05-09 14:29:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962663002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962663002/1

[commit-bot: I haz the power, 2016-05-09 14:32:33]

Description was changed from

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org
==========

to

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org
==========

[commit-bot: I haz the power, 2016-05-09 14:32:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-09 14:33:33]

Description was changed from

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org
==========

to

==========
[fullcodegen] Factor out VisitProperty from architectures.

This makes the aforementioned visitation function independent of the
target architecture by leveraging existing abstractions.

R=mvstanton@chromium.org

Committed: https://crrev.com/d0e84c8ebd43b3da41e4154f349f42de60448cb2
Cr-Commit-Position: refs/heads/master@{#36111}
==========

[commit-bot: I haz the power, 2016-05-09 14:33:34]

Patchset 1 (id:??) landed as
https://crrev.com/d0e84c8ebd43b3da41e4154f349f42de60448cb2
Cr-Commit-Position: refs/heads/master@{#36111}