Skip to content

Conversation

kazutakahirata
Copy link
Contributor

We do use MappingTraits in this file, but all uses are inside the
llvm::yaml namespace, so we don't rely on the "using" decl.

We do use MappingTraits in this file, but all uses are inside the
llvm::yaml namespace, so we don't rely on the "using" decl.
@llvmbot
Copy link
Member

llvmbot commented May 17, 2025

@llvm/pr-subscribers-clang-tools-extra

Author: Kazu Hirata (kazutakahirata)

Changes

We do use MappingTraits in this file, but all uses are inside the
llvm::yaml namespace, so we don't rely on the "using" decl.


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

1 Files Affected:

  • (modified) clang-tools-extra/clang-include-fixer/find-all-symbols/SymbolInfo.cpp (-1)
diff --git a/clang-tools-extra/clang-include-fixer/find-all-symbols/SymbolInfo.cpp b/clang-tools-extra/clang-include-fixer/find-all-symbols/SymbolInfo.cpp
index 4a5f8353b4105..3ac3f73b6cb9a 100644
--- a/clang-tools-extra/clang-include-fixer/find-all-symbols/SymbolInfo.cpp
+++ b/clang-tools-extra/clang-include-fixer/find-all-symbols/SymbolInfo.cpp
@@ -12,7 +12,6 @@
 #include "llvm/Support/YAMLTraits.h"
 #include "llvm/Support/raw_ostream.h"
 
-using llvm::yaml::MappingTraits;
 using ContextType = clang::find_all_symbols::SymbolInfo::ContextType;
 using clang::find_all_symbols::SymbolInfo;
 using clang::find_all_symbols::SymbolAndSignals;

@kazutakahirata kazutakahirata requested a review from kadircet May 17, 2025 05:07
@kazutakahirata kazutakahirata merged commit c0ca030 into llvm:main May 18, 2025
13 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_001_tidy_misc-unused-using-decls_clang_include_fixer branch May 18, 2025 00:00
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