Apply rule-of-three

Applies rule-of-three by removing missing copy ctors.

Author: smilczek

IGC/AdaptorOCL/CLElfLib/ElfReader.h

@@ -51,6 +51,9 @@ class CElfReader {
 
   ELF_CALL ~CElfReader();
 
+  CElfReader(const CElfReader &) = delete;
+  CElfReader &operator=(const CElfReader &) = delete;
+
   SElfHeader *m_pElfHeader; // pointer to the ELF header
   const char *m_pBinary;    // portable ELF binary
   char *m_pNameTable;       // pointer to the string table

IGC/AdaptorOCL/OCL/util/BinaryStream.h

@@ -17,6 +17,9 @@ class BinaryStream {
   BinaryStream();
   ~BinaryStream();
 
+  BinaryStream(const BinaryStream &) = delete;
+  BinaryStream &operator=(const BinaryStream &) = delete;
+
   bool Write(const char *s, std::streamsize n);
 
   bool Write(const BinaryStream &in);

IGC/BiFManager/BiFManagerCommon.hpp

@@ -35,6 +35,9 @@ class CollectBuiltinsPass : public llvm::InstVisitor<CollectBuiltinsPass> {
   CollectBuiltinsPass(TFunctionsVec &neededBuiltinsFunc, const std::function<bool(llvm::Function *)> &predicate);
   ~CollectBuiltinsPass();
 
+  CollectBuiltinsPass(const CollectBuiltinsPass &) = delete;
+  CollectBuiltinsPass &operator=(const CollectBuiltinsPass &) = delete;
+
   void visitCallInst(llvm::CallInst &callInst);
 };
 
@@ -60,6 +63,9 @@ class BiFManagerCommon {
   BiFManagerCommon(llvm::LLVMContext &Context);
   ~BiFManagerCommon();
 
+  BiFManagerCommon(const BiFManagerCommon &) = delete;
+  BiFManagerCommon &operator=(const BiFManagerCommon &) = delete;
+
   static size_t getHash(const std::string &FlagName);
 
 protected:

IGC/BiFManager/BiFManagerHandler.hpp

@@ -32,6 +32,9 @@ class BiFManagerHandler : public BiFManagerCommon {
   BiFManagerHandler(llvm::LLVMContext &Context);
   ~BiFManagerHandler();
 
+  BiFManagerHandler(const BiFManagerHandler &) = delete;
+  BiFManagerHandler &operator=(const BiFManagerHandler &) = delete;
+
   // Indicator if we have pointer size 32 in user module
   bool isPtrSizeInBits32;
 

IGC/Compiler/CISACodeGen/ComputeShaderBase.hpp

@@ -20,6 +20,9 @@ class CComputeShaderBase : public CShader {
   CComputeShaderBase(llvm::Function *pFunc, CShaderProgram *pProgram, GenericShaderState &GState);
   virtual ~CComputeShaderBase();
 
+  CComputeShaderBase(const CComputeShaderBase &) = delete;
+  CComputeShaderBase &operator=(const CComputeShaderBase &) = delete;
+
 protected:
   // Determines if HW can handle auto generating local IDs with this
   // order

IGC/Compiler/CISACodeGen/DebugInfo.hpp

@@ -34,6 +34,10 @@ class DebugInfoPass : public llvm::ModulePass {
   DebugInfoPass(CShaderProgram::KernelShaderMap &);
   virtual llvm::StringRef getPassName() const override { return "DebugInfoPass"; }
   virtual ~DebugInfoPass();
+
+  DebugInfoPass(const DebugInfoPass &) = delete;
+  DebugInfoPass &operator=(const DebugInfoPass &) = delete;
+
   static char ID;
 
 private:
@@ -57,6 +61,10 @@ class CatchAllLineNumber : public llvm::FunctionPass {
 public:
   CatchAllLineNumber();
   virtual ~CatchAllLineNumber();
+
+  CatchAllLineNumber(const CatchAllLineNumber &) = delete;
+  CatchAllLineNumber &operator=(const CatchAllLineNumber &) = delete;
+
   static char ID;
 
   llvm::StringRef getPassName() const override { return "CatchAllLineNumber"; }

IGC/Compiler/CISACodeGen/EmitVISAPass.hpp

@@ -55,6 +55,9 @@ class EmitPass : public llvm::FunctionPass {
 
   virtual ~EmitPass();
 
+  EmitPass(const EmitPass &) = delete;
+  EmitPass &operator=(const EmitPass &) = delete;
+
   // Note:  all analysis passes should be function passes. If a module analysis pass
   //        is used, it would invalidate function analysis passes and therefore cause
   //        those analysis passes to be invoked twice, which increases compiling time.

IGC/Compiler/CISACodeGen/GenCodeGenModule.h

@@ -40,6 +40,10 @@ class GenXCodeGenModule : public llvm::ModulePass {
   static char ID;
   GenXCodeGenModule();
   ~GenXCodeGenModule();
+
+  GenXCodeGenModule(const GenXCodeGenModule &) = delete;
+  GenXCodeGenModule &operator=(const GenXCodeGenModule &) = delete;
+
   virtual llvm::StringRef getPassName() const override { return "GenX CodeGen module"; }
   void getAnalysisUsage(llvm::AnalysisUsage &AU) const override;
   bool runOnModule(llvm::Module &M) override;

IGC/Compiler/Optimizer/OpenCLPasses/UndefinedReferences/UndefinedReferencesPass.hpp

@@ -22,8 +22,6 @@ class UndefinedReferencesPass : public llvm::ModulePass {
 
   UndefinedReferencesPass();
 
-  ~UndefinedReferencesPass() {}
-
   virtual llvm::StringRef getPassName() const override { return "UndefinedReferencesPass"; }
   virtual void getAnalysisUsage(llvm::AnalysisUsage &AU) const override { AU.addRequired<CodeGenContextWrapper>(); }