Skip to content

Commit 6a2055e

Browse files
smilczekigcbot
authored andcommitted
Apply rule-of-three
Apply rule-of-three by explicitly deleting copy ctors.
1 parent 0b5f9e1 commit 6a2055e

File tree

9 files changed

+8
-12
lines changed

9 files changed

+8
-12
lines changed

visa/Attributes.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,6 @@ class Attributes {
121121
}
122122

123123
Attributes();
124-
~Attributes() {}
125124

126125
// Set attribute's value
127126
void setKernelAttr(ID kID, bool v);

visa/BitSet.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -436,6 +436,8 @@ class SparseBitSet {
436436
SparseBitSet(const SparseBitSet &&Other)
437437
: Segments(std::move(Other.Segments)), MaxBits(Other.MaxBits) {}
438438

439+
~SparseBitSet() = default;
440+
439441
unsigned getSize() const { return MaxBits; }
440442

441443
void clear() { Segments.clear(); }

visa/FrequencyInfo.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ class FrequencyInfo {
5959
void sortBasedOnFreq(std::vector<LiveRange *> &lrs);
6060
bool hasFreqMetaData(G4_INST *i);
6161
void deriveRefFreq(G4_BB *bb);
62-
~FrequencyInfo() {}
6362
void dump() const {}
6463
void initForRegAlloc(LivenessAnalysis *l);
6564
void initGRFSpillFillFreq() {

visa/GraphColor.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1312,7 +1312,6 @@ class ForbiddenRegs {
13121312
forbiddenVec[(size_t)forbiddenKind::FBD_SCALAR].resize(
13131313
getForbiddenVectorSize(G4_SCALAR));
13141314
};
1315-
~ForbiddenRegs(){};
13161315

13171316
unsigned getForbiddenVectorSize(G4_RegFileKind regKind) const;
13181317
void generateReservedGRFForbidden(unsigned reserveSpillSize);

visa/Mem_Manager.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ class Mem_Manager {
2222
Mem_Manager(size_t defaultArenaSize);
2323
~Mem_Manager();
2424

25+
Mem_Manager(const Mem_Manager &) = delete;
26+
Mem_Manager &operator=(const Mem_Manager &) = delete;
27+
2528
void *alloc(size_t size) {
2629
return _arenaManager.AllocDataSpace(size, ArenaHeader::defaultAlign);
2730
}

visa/Option.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -332,8 +332,6 @@ class Options {
332332
}
333333
explicit VISAOptionsDB(Options *opt)
334334
: options(opt), optionsMap(static_cast<int>(vISA_NUM_OPTIONS)) {}
335-
336-
~VISAOptionsDB() {}
337335
};
338336

339337
VISAOptionsDB m_vISAOptions;

visa/RPE.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@ class RPE {
1818
RPE(const GlobalRA &, const LivenessAnalysis *,
1919
DECLARE_LIST *spills = nullptr);
2020

21-
~RPE() {}
22-
2321
void run();
2422
void runBB(G4_BB *);
2523
unsigned int getRegisterPressure(G4_INST *inst) {

visa/SpillManagerGMRF.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -253,8 +253,6 @@ class SpillManagerGRF {
253253
const LivenessAnalysis *lvInfo, LSLR_LIST *spilledLSLRs,
254254
bool enableSpillSpaceCompression, bool useScratchMsg);
255255

256-
~SpillManagerGRF() {}
257-
258256
bool insertSpillFillCode(G4_Kernel *kernel,
259257
PointsToAnalysis &pointsToAnalysis);
260258

visa/iga/IGALibrary/MemManager/MemManager.hpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ class MemManager {
2222
MemManager(size_t defaultArenaSize);
2323
~MemManager();
2424

25+
MemManager(const MemManager &) = delete;
26+
MemManager &operator=(const MemManager &) = delete;
27+
2528
void *alloc(size_t size) { return _arenaManager.AllocDataSpace(size); }
2629

2730
private:
2831
ArenaManager _arenaManager;
29-
30-
MemManager(const MemManager &) = delete;
31-
MemManager &operator=(const MemManager &) = delete;
3232
};
3333
} // namespace iga
3434
#endif

0 commit comments

Comments
 (0)