Skip to content

Commit 068944c

Browse files
committed
[llvm] manual fix-ups to IDS codemod of DebugInfo library
1 parent 41fcfb0 commit 068944c

File tree

6 files changed

+16
-8
lines changed

6 files changed

+16
-8
lines changed

llvm/include/llvm/DebugInfo/CodeView/ContinuationRecordBuilder.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
#include "llvm/ADT/SmallVector.h"
1515
#include "llvm/DebugInfo/CodeView/CVRecord.h"
1616
#include "llvm/DebugInfo/CodeView/TypeRecordMapping.h"
17+
#include "llvm/DebugInfo/CodeView/TypeRecord.h"
1718
#include "llvm/Support/BinaryByteStream.h"
1819
#include "llvm/Support/BinaryStreamWriter.h"
1920
#include <cstdint>

llvm/include/llvm/DebugInfo/CodeView/SimpleTypeSerializer.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111

1212
#include "llvm/Support/Compiler.h"
1313
#include "llvm/ADT/ArrayRef.h"
14+
#include "llvm/DebugInfo/CodeView/TypeRecord.h"
1415
#include <vector>
1516

1617
namespace llvm {
@@ -32,7 +33,6 @@ class SimpleTypeSerializer {
3233
// Don't allow serialization of field list records using this interface.
3334
ArrayRef<uint8_t> serialize(const FieldListRecord &Record) = delete;
3435
};
35-
3636
} // end namespace codeview
3737
} // end namespace llvm
3838

llvm/include/llvm/DebugInfo/LogicalView/Core/LVSort.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
#ifndef LLVM_DEBUGINFO_LOGICALVIEW_CORE_LVSORT_H
1414
#define LLVM_DEBUGINFO_LOGICALVIEW_CORE_LVSORT_H
1515

16+
#include "llvm/Support/Compiler.h"
17+
1618
namespace llvm {
1719
namespace logicalview {
1820

llvm/include/llvm/DebugInfo/PDB/UDTLayout.h

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,10 @@
1515
#include "llvm/ADT/StringRef.h"
1616
#include "llvm/DebugInfo/PDB/PDBSymbol.h"
1717
#include "llvm/DebugInfo/PDB/PDBSymbolData.h"
18+
#include "llvm/DebugInfo/PDB/PDBSymbolFunc.h"
1819
#include "llvm/DebugInfo/PDB/PDBSymbolTypeBaseClass.h"
1920
#include "llvm/DebugInfo/PDB/PDBSymbolTypeBuiltin.h"
21+
#include "llvm/DebugInfo/PDB/PDBSymbolTypeFunctionSig.h"
2022
#include "llvm/DebugInfo/PDB/PDBSymbolTypeUDT.h"
2123
#include "llvm/DebugInfo/PDB/PDBSymbolTypeVTable.h"
2224
#include <cstdint>
@@ -115,6 +117,10 @@ class LLVM_ABI UDTLayoutBase : public LayoutItemBase {
115117
const std::string &Name, uint32_t OffsetInParent, uint32_t Size,
116118
bool IsElided);
117119

120+
// Explicitly non-copyable.
121+
UDTLayoutBase(UDTLayoutBase const&) = delete;
122+
UDTLayoutBase& operator=(UDTLayoutBase const&) = delete;
123+
118124
uint32_t tailPadding() const override;
119125
ArrayRef<LayoutItemBase *> layout_items() const { return LayoutItems; }
120126
ArrayRef<BaseClassLayout *> bases() const { return AllBases; }
@@ -164,8 +170,6 @@ class LLVM_ABI ClassLayout : public UDTLayoutBase {
164170
explicit ClassLayout(const PDBSymbolTypeUDT &UDT);
165171
explicit ClassLayout(std::unique_ptr<PDBSymbolTypeUDT> UDT);
166172

167-
ClassLayout(ClassLayout &&Other) = default;
168-
169173
const PDBSymbolTypeUDT &getClass() const { return UDT; }
170174
uint32_t immediatePadding() const override;
171175

llvm/lib/DebugInfo/CodeView/ContinuationRecordBuilder.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -244,8 +244,8 @@ std::vector<CVType> ContinuationRecordBuilder::end(TypeIndex Index) {
244244
// implement this in the cpp file.
245245
#define TYPE_RECORD(EnumName, EnumVal, Name)
246246
#define TYPE_RECORD_ALIAS(EnumName, EnumVal, Name, AliasName)
247-
#define MEMBER_RECORD(EnumName, EnumVal, Name) \
248-
template void llvm::codeview::ContinuationRecordBuilder::writeMemberType( \
247+
#define MEMBER_RECORD(EnumName, EnumVal, Name) \
248+
template LLVM_ABI void llvm::codeview::ContinuationRecordBuilder::writeMemberType( \
249249
Name##Record &Record);
250250
#define MEMBER_RECORD_ALIAS(EnumName, EnumVal, Name, AliasName)
251251
#include "llvm/DebugInfo/CodeView/CodeViewTypes.def"

llvm/lib/DebugInfo/CodeView/SimpleTypeSerializer.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
#include "llvm/DebugInfo/CodeView/RecordSerialization.h"
1212
#include "llvm/DebugInfo/CodeView/TypeRecordMapping.h"
1313
#include "llvm/Support/BinaryStreamWriter.h"
14+
#include "llvm/Support/Compiler.h"
1415

1516
using namespace llvm;
1617
using namespace llvm::codeview;
@@ -59,9 +60,9 @@ ArrayRef<uint8_t> SimpleTypeSerializer::serialize(T &Record) {
5960

6061
// Explicitly instantiate the member function for each known type so that we can
6162
// implement this in the cpp file.
62-
#define TYPE_RECORD(EnumName, EnumVal, Name) \
63-
template ArrayRef<uint8_t> llvm::codeview::SimpleTypeSerializer::serialize( \
64-
Name##Record &Record);
63+
#define TYPE_RECORD(EnumName, EnumVal, Name) \
64+
template LLVM_ABI ArrayRef<uint8_t> \
65+
llvm::codeview::SimpleTypeSerializer::serialize(Name##Record &Record);
6566
#define TYPE_RECORD_ALIAS(EnumName, EnumVal, Name, AliasName)
6667
#define MEMBER_RECORD(EnumName, EnumVal, Name)
6768
#define MEMBER_RECORD_ALIAS(EnumName, EnumVal, Name, AliasName)

0 commit comments

Comments
 (0)