[llvm] 87c5437 - PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 6 03:28:56 PDT 2020


Author: Simon Pilgrim
Date: 2020-08-06T11:28:42+01:00
New Revision: 87c5437afd273e909e0fed3389de7531d5452ea5

URL: https://github.com/llvm/llvm-project/commit/87c5437afd273e909e0fed3389de7531d5452ea5
DIFF: https://github.com/llvm/llvm-project/commit/87c5437afd273e909e0fed3389de7531d5452ea5.diff

LOG: PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI.

We already need to include raw_ostream.h, also add missing StringRef.h and cstdint implicit dependencies.

Remove unnecessary includes from PDBExtras.cpp

Added: 
    

Modified: 
    llvm/include/llvm/DebugInfo/PDB/PDBExtras.h
    llvm/lib/DebugInfo/PDB/PDBExtras.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/DebugInfo/PDB/PDBExtras.h b/llvm/include/llvm/DebugInfo/PDB/PDBExtras.h
index 45aba013e7c8..0a10ffcb309b 100644
--- a/llvm/include/llvm/DebugInfo/PDB/PDBExtras.h
+++ b/llvm/include/llvm/DebugInfo/PDB/PDBExtras.h
@@ -9,16 +9,15 @@
 #ifndef LLVM_DEBUGINFO_PDB_PDBEXTRAS_H
 #define LLVM_DEBUGINFO_PDB_PDBEXTRAS_H
 
+#include "llvm/ADT/StringRef.h"
 #include "llvm/DebugInfo/CodeView/CodeView.h"
 #include "llvm/DebugInfo/PDB/PDBTypes.h"
 #include "llvm/Support/raw_ostream.h"
-
 #include <unordered_map>
+#include <cstdint>
 
 namespace llvm {
 
-class raw_ostream;
-
 namespace pdb {
 
 using TagStats = std::unordered_map<PDB_SymType, int>;
@@ -51,7 +50,6 @@ void dumpSymbolField(raw_ostream &OS, StringRef Name, T Value, int Indent) {
   OS << Name << ": " << Value;
 }
 
-
 } // end namespace pdb
 
 } // end namespace llvm

diff  --git a/llvm/lib/DebugInfo/PDB/PDBExtras.cpp b/llvm/lib/DebugInfo/PDB/PDBExtras.cpp
index 354a99476c4b..8057e861f331 100644
--- a/llvm/lib/DebugInfo/PDB/PDBExtras.cpp
+++ b/llvm/lib/DebugInfo/PDB/PDBExtras.cpp
@@ -7,8 +7,6 @@
 //===----------------------------------------------------------------------===//
 
 #include "llvm/DebugInfo/PDB/PDBExtras.h"
-#include "llvm/ADT/ArrayRef.h"
-#include "llvm/Support/raw_ostream.h"
 
 using namespace llvm;
 using namespace llvm::pdb;
@@ -140,6 +138,10 @@ raw_ostream &llvm::pdb::operator<<(raw_ostream &OS,
     OS << #name;                                                               \
     return OS;
 #include "llvm/DebugInfo/CodeView/CodeViewRegisters.def"
+#include "llvm/DebugInfo/CodeView/CodeView.h"             // for RegisterId, ThunkOrdinal, CPURegister, CPUType, CPUType::ARM64, ThunkOrdinal::BranchIsland, ThunkOrdinal::Pcode, ThunkOrdinal::Standard, ThunkOrdinal::ThisAdjustor, ThunkOrdinal::TrampIncremental, ThunkOrdinal::UnknownLoad, ThunkOrdinal::Vcall
+#include "llvm/DebugInfo/PDB/PDBTypes.h"                  // for PDB_SymType, PDB_CallingConv, PDB_VariantType, PDB_BuiltinType, PDB_Machine, PDB_Lang, Variant, PDB_LocType, Variant::(anonymous), PDB_DataKind, PDB_Checksum, PDB_SourceCompression, PDB_UdtType, PDB_MemberAccess, VersionInfo, PDB_VariantType::Bool, PDB_VariantType::Double, PDB_VariantType::Int16, PDB_VariantType::Int32, PDB_VariantType::Int64, PDB_VariantType::Int8, PDB_VariantType::Single, PDB_VariantType::UInt16, PDB_VariantType::UInt32, PDB_VariantType::UInt64, PDB_VariantType::UInt8, PDB_BuiltinType::BCD, PDB_BuiltinType::BSTR, PDB_BuiltinType::Bitfield, PDB_BuiltinType::Bool, PDB_BuiltinType::Char, PDB_BuiltinType::Char16, PDB_BuiltinType::Char32, PDB_BuiltinType::Complex, PDB_BuiltinType::Currency, PDB_BuiltinType::Date, PDB_BuiltinType::Float, PDB_BuiltinType::HResult, PDB_BuiltinType::Int, PDB_BuiltinType::Long, PDB_BuiltinType::None, PDB_BuiltinType::UInt, PDB_BuiltinType::ULong, PDB_BuiltinType::Variant, PDB_BuiltinType::Void, PDB_...
+#include "llvm/Support/raw_ostream.h"                     // for raw_ostream
+#include "stdint.h"                                       // for uint32_t
 #undef CV_REGISTER
 #undef CV_REGISTERS_X86
 


        


More information about the llvm-commits mailing list