fix review comments

src/CMakeLists.txt

@@ -12,34 +12,34 @@
 # limitations under the License.
 
 set (SRC
-    tree.cpp
-    util.cpp
     log.cpp
     ngraph/descriptor/input.cpp
     ngraph/descriptor/output.cpp
-    ngraph/descriptor/tensor.cpp
     ngraph/descriptor/tensor_view.cpp
+    ngraph/descriptor/tensor.cpp
+    ngraph/node.cpp
+    ngraph/pass/call_pass.cpp
+    ngraph/pass/manager.cpp
+    ngraph/pass/pass.cpp
+    ngraph/pass/propagate_types.cpp
+    ngraph/pass/topological_sort.cpp
+    ngraph/pass/tree_pass.cpp
+    ngraph/visualize.cpp
     ops/binary_elementwise_builtin.cpp
     ops/broadcast.cpp
     ops/concatenate.cpp
-    ops/convert.cpp
     ops/constant.cpp
+    ops/convert.cpp
     ops/dot.cpp
     ops/function.cpp
     ops/op.cpp
     ops/parameter.cpp
     ops/tuple.cpp
     ops/unary_elementwise_builtin.cpp
+    tree.cpp
     types/element_type.cpp
     types/type.cpp
-    ngraph/node.cpp
-    ngraph/visualize.cpp
-    ngraph/pass/pass.cpp
-    ngraph/pass/manager.cpp
-    ngraph/pass/call_pass.cpp
-    ngraph/pass/tree_pass.cpp
-    ngraph/pass/topological_sort.cpp
-    ngraph/pass/propagate_types.cpp
+    util.cpp
     )
 
 set(NGRAPH_INCLUDE_PATH

test/CMakeLists.txt

@@ -22,19 +22,19 @@ include_directories(
     )
 
 set (SRC
-    main.cpp
     build_graph.cpp
     eigen.cpp
     element_type.cpp
-    op.cpp
     input_output_assign.cpp
+    main.cpp
+    op.cpp
+    pass_manager.cpp
     tensor.cpp
+    test_tools.cpp
     topological_sort.cpp
     type_prop.cpp
     util.cpp
     uuid.cpp
-    pass_manager.cpp
-    test_tools.cpp
 )
 
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")

test/test_tools.cpp

@@ -21,6 +21,8 @@
 using namespace std;
 using namespace ngraph;
 
+// This function traverses the list of ops and verifies that each op's dependencies (its inputs)
+// is located earlier in the list. That is enough to be valid
 bool validate_list(const list<Node*>& nodes)
 {
     bool rc = true;