https://codereview.chromium.org/768763002/diff/1/src/compiler/graph.h
File src/compiler/graph.h (right):
https://codereview.chromium.org/768763002/diff/1/src/compiler/graph.h#newcode22
src/compiler/graph.h:22: template <typename LocalState>
nit: Can we use s/LocalState/State/ here for readability?
https://codereview.chromium.org/768763002/diff/1/src/compiler/graph.h#newcode23
src/compiler/graph.h:23: class GraphTraversal : public ZoneObject {
As discussed offline: Since this is not really a "traversal" but just
the "state for a traversal", how about calling it NodeState? This would
also warrant moving it into the node.h file.
https://codereview.chromium.org/768763002/diff/1/src/compiler/graph.h#newcode115
src/compiler/graph.h:115: template <typename LocalState>
nit: Can we use s/LocalState/State/ here for readability?
https://codereview.chromium.org/768763002/
--
--
v8-dev mailing list
[email protected]
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 an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
