Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

This patch simplifies DOTGraphTraits with an inherited constructor.
This way, we do not need to repeat the constructor signature or
manually forward constructor arguments.

This patch simplifies DOTGraphTraits with an inherited constructor.
This way, we do not need to repeat the constructor signature or
manually forward constructor arguments.
@llvmbot
Copy link
Member

llvmbot commented Oct 18, 2025

@llvm/pr-subscribers-llvm-support

Author: Kazu Hirata (kazutakahirata)

Changes

This patch simplifies DOTGraphTraits with an inherited constructor.
This way, we do not need to repeat the constructor signature or
manually forward constructor arguments.


Full diff: https://github.com/llvm/llvm-project/pull/164056.diff

1 Files Affected:

  • (modified) llvm/include/llvm/Support/DOTGraphTraits.h (+1-1)
diff --git a/llvm/include/llvm/Support/DOTGraphTraits.h b/llvm/include/llvm/Support/DOTGraphTraits.h
index ffa9abe328c83..bf30aa4df9fab 100644
--- a/llvm/include/llvm/Support/DOTGraphTraits.h
+++ b/llvm/include/llvm/Support/DOTGraphTraits.h
@@ -164,7 +164,7 @@ struct DefaultDOTGraphTraits {
 ///
 template <typename Ty>
 struct DOTGraphTraits : public DefaultDOTGraphTraits {
-  DOTGraphTraits (bool simple=false) : DefaultDOTGraphTraits (simple) {}
+  using DefaultDOTGraphTraits::DefaultDOTGraphTraits;
 };
 
 } // End llvm namespace

@kazutakahirata kazutakahirata merged commit df6d5f2 into llvm:main Oct 18, 2025
12 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20251017_Support_DOTGraphTraits branch October 18, 2025 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants