[llvm] [SandboxVec][DAG] Cleanup: Move callback registration from Scheduler to DAG (PR #116455)

Jorge Gorbe Moya via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 18:02:32 PST 2024


================
@@ -325,9 +328,26 @@ class DependencyGraph {
   /// chain.
   void createNewNodes(const Interval<Instruction> &NewInterval);
 
+  /// Called by the callbacks when a new instruction \p I has been created.
+  void notifyCreateInstr(Instruction *I) {
+    getOrCreateNode(I);
+    // TODO: Update the dependencies for the new node.
+    // TODO: Update the MemDGNode chain to include the new node if needed.
+  }
+
 public:
   DependencyGraph(AAResults &AA)
----------------
slackito wrote:

What's this constructor used for? Is it a temporary thing while the other pieces fall into place or does it make sense to create a DAG without a sandbox IR context?

https://github.com/llvm/llvm-project/pull/116455


More information about the llvm-commits mailing list