From cae5a9314423fcd4e1b21e8befe4e022002c9b38 Mon Sep 17 00:00:00 2001
From: superjomn <yanchunwei@outlook.com>
Date: Wed, 17 Apr 2019 20:32:26 +0800
Subject: [PATCH] add topological sort

---
 .../lite/core/mir/generate_program_pass.cc    | 11 +++++++-
 .../lite/core/mir/generate_program_pass.h     |  9 ++++++-
 paddle/fluid/lite/core/mir/ssa_graph.cc       |  6 +++++
 paddle/fluid/lite/core/mir/ssa_graph.h        | 27 ++++++++++++++++++-
 4 files changed, 50 insertions(+), 3 deletions(-)

diff --git a/paddle/fluid/lite/core/mir/generate_program_pass.cc b/paddle/fluid/lite/core/mir/generate_program_pass.cc
index 0be516489d..659a959fec 100644
--- a/paddle/fluid/lite/core/mir/generate_program_pass.cc
+++ b/paddle/fluid/lite/core/mir/generate_program_pass.cc
@@ -18,7 +18,16 @@
 namespace paddle {
 namespace lite {
 namespace mir {
-void GenerateProgramPass::Apply(std::unique_ptr<mir::SSAGraph> &graph) {}
+
+void GenerateProgramPass::Apply(std::unique_ptr<mir::SSAGraph>& graph) {
+  for (auto& item : graph->TopoloticalOrder()) {
+    if (item->IsInstruct()) {
+      auto& instruct = item->AsInstruct();
+      kernels_.emplace_back(std::move(instruct.valid_kernels.front()));
+    }
+  }
+}
+
 }  // namespace mir
 }  // namespace lite
 }  // namespace paddle
diff --git a/paddle/fluid/lite/core/mir/generate_program_pass.h b/paddle/fluid/lite/core/mir/generate_program_pass.h
index bc78370b08..0421f5f2ad 100644
--- a/paddle/fluid/lite/core/mir/generate_program_pass.h
+++ b/paddle/fluid/lite/core/mir/generate_program_pass.h
@@ -14,6 +14,8 @@
 
 #pragma once
 
+#include <list>
+#include "paddle/fluid/lite/core/kernel.h"
 #include "paddle/fluid/lite/core/mir/pass.h"
 
 namespace paddle {
@@ -26,7 +28,12 @@ namespace mir {
  */
 class GenerateProgramPass : public ProgramPass {
  public:
-  void Apply(std::unique_ptr<mir::SSAGraph> &graph) override;
+  void Apply(std::unique_ptr<mir::SSAGraph>& graph) override;
+
+  std::list<std::unique_ptr<KernelBase>>& kernels() { return kernels_; }
+
+ private:
+  std::list<std::unique_ptr<KernelBase>> kernels_;
 };
 
 }  // namespace mir
diff --git a/paddle/fluid/lite/core/mir/ssa_graph.cc b/paddle/fluid/lite/core/mir/ssa_graph.cc
index 54d570cab4..cfab5b35f0 100644
--- a/paddle/fluid/lite/core/mir/ssa_graph.cc
+++ b/paddle/fluid/lite/core/mir/ssa_graph.cc
@@ -13,3 +13,9 @@
 // limitations under the License.
 
 #include "paddle/fluid/lite/core/mir/ssa_graph.h"
+
+namespace paddle {
+namespace lite {
+namespace mir {}  // namespace mir
+}  // namespace lite
+}  // namespace paddle
diff --git a/paddle/fluid/lite/core/mir/ssa_graph.h b/paddle/fluid/lite/core/mir/ssa_graph.h
index 6dee98c76d..95d2d2d98d 100644
--- a/paddle/fluid/lite/core/mir/ssa_graph.h
+++ b/paddle/fluid/lite/core/mir/ssa_graph.h
@@ -16,6 +16,7 @@
 
 #include <list>
 #include <map>
+#include <stack>
 #include <string>
 #include <vector>
 #include "paddle/fluid/lite/core/mir/node.h"
@@ -81,7 +82,31 @@ class SSAGraph : GraphBase {
     }
   }
 
-  std::vector<mir::Node *> TopoloticalOrder() const;
+  void sort_utils(mir::Node *n, std::map<mir::Node *, bool> &visited,
+                  std::stack<mir::Node *> &stack) {
+    visited[n] = true;
+    for (auto &out : n->outlinks) {
+      if (!visited[out]) {
+        sort_utils(out, visited, stack);
+      }
+    }
+  }
+
+  std::vector<mir::Node *> TopoloticalOrder() {
+    std::map<mir::Node *, bool> visited;
+    std::stack<mir::Node *> stack;
+    std::vector<mir::Node *> res;
+
+    for (auto &n : mutable_nodes()) {
+      if (!visited[&n]) sort_utils(&n, visited, stack);
+    }
+
+    while (!stack.empty()) {
+      res.push_back(stack.top());
+      stack.pop();
+    }
+    return res;
+  }
 
   const std::list<mir::Node> &nodes() const { return node_storage_; }
   std::list<mir::Node> &mutable_nodes() { return node_storage_; }
-- 
GitLab