[llvm] [LLVM] Skip dumping inline SDag children (PR #141359)

Jon Roelofs via llvm-commits llvm-commits at lists.llvm.org
Sat May 24 09:49:41 PDT 2025


https://github.com/jroelofs created https://github.com/llvm/llvm-project/pull/141359

If they're simple enough to render inline, we don't need to dump them again in the recursive walk.

>From 3062597c202bf95cb6b5a8d7d9f1326b84e32845 Mon Sep 17 00:00:00 2001
From: Jon Roelofs <jonathan_roelofs at apple.com>
Date: Sat, 24 May 2025 09:46:08 -0700
Subject: [PATCH] [LLVM] Skip dumping inline SDag children

If they're simple enough to render inline, we don't need to dump them again in
the recursive walk.
---
 llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
index ee4297f3ad8ce..61c6a9470e531 100644
--- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
@@ -1165,6 +1165,9 @@ static void printrWithDepthHelper(raw_ostream &OS, const SDNode *N,
     // Don't follow chain operands.
     if (Op.getValueType() == MVT::Other)
       continue;
+    // Don't print children that were fully rendered inline.
+    if (shouldPrintInline(*Op.getNode(), G))
+      continue;
     OS << '\n';
     printrWithDepthHelper(OS, Op.getNode(), G, depth - 1, indent + 2);
   }



More information about the llvm-commits mailing list