From: Craig Topper <craig.topper@intel.com>
Date: Sun, 18 Mar 2018 21:28:11 +0000 (+0000)
Subject: [SelectionDAG] Don't default the SelectionDAG* parameter to SDValue::dump to nullptr... 
X-Git-Tag: llvmorg-7.0.0-rc1~10325
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f7fdfb3fb6318fd032e9335a4f40275c7da1b1c2;p=platform%2Fupstream%2Fllvm.git

[SelectionDAG] Don't default the SelectionDAG* parameter to SDValue::dump to nullptr. Use two different signatures instead.

This matches what we do in SDNode.

This should allow SDValue::dump to be used in the debugger without getting an error if you don't pass an argument.

llvm-svn: 327811
---

diff --git a/llvm/include/llvm/CodeGen/SelectionDAGNodes.h b/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
index 557d939..ebca251 100644
--- a/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
+++ b/llvm/include/llvm/CodeGen/SelectionDAGNodes.h
@@ -189,8 +189,10 @@ public:
   inline bool isUndef() const;
   inline unsigned getMachineOpcode() const;
   inline const DebugLoc &getDebugLoc() const;
-  inline void dump(const SelectionDAG *G = nullptr) const;
-  inline void dumpr(const SelectionDAG *G = nullptr) const;
+  inline void dump() const;
+  inline void dump(const SelectionDAG *G) const;
+  inline void dumpr() const;
+  inline void dumpr(const SelectionDAG *G) const;
 
   /// Return true if this operand (which must be a chain) reaches the
   /// specified operand without crossing any side-effecting instructions.
@@ -1093,10 +1095,18 @@ inline const DebugLoc &SDValue::getDebugLoc() const {
   return Node->getDebugLoc();
 }
 
+inline void SDValue::dump() const {
+  return Node->dump();
+}
+
 inline void SDValue::dump(const SelectionDAG *G) const {
   return Node->dump(G);
 }
 
+inline void SDValue::dumpr() const {
+  return Node->dumpr();
+}
+
 inline void SDValue::dumpr(const SelectionDAG *G) const {
   return Node->dumpr(G);
 }