Remove unnecessary semicolons (#2071)

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@b045a91

Author: svenvh

llvm-spirv/lib/SPIRV/Mangler/ParameterType.h

@@ -153,10 +153,10 @@ struct TypeVisitor;
 struct ParamType {
   /// @brief Constructor.
   /// @param TypeEnum type id.
-  ParamType(TypeEnum TypeId) : TypeId(TypeId){};
+  ParamType(TypeEnum TypeId) : TypeId(TypeId) {}
 
   /// @brief Destructor.
-  virtual ~ParamType(){};
+  virtual ~ParamType() {}
 
   /// Abstract Methods ///
 

llvm-spirv/lib/SPIRV/libSPIRV/SPIRVDecorate.h

@@ -371,7 +371,7 @@ class SPIRVDecorationGroup : public SPIRVEntry {
   SPIRVDecorationGroup(SPIRVModule *TheModule, SPIRVId TheId)
       : SPIRVEntry(TheModule, WC, OC, TheId) {
     validate();
-  };
+  }
   // Incomplete constructor
   SPIRVDecorationGroup() : SPIRVEntry(OC) {}
   void encodeAll(spv_ostream &O) const override;
@@ -631,12 +631,12 @@ class SPIRVDecorateFunctionRoundingModeINTEL : public SPIRVDecorate {
                                          SPIRVWord TargetWidth,
                                          spv::FPRoundingMode FloatControl)
       : SPIRVDecorate(spv::DecorationFunctionRoundingModeINTEL, TheTarget,
-                      TargetWidth, static_cast<SPIRVWord>(FloatControl)){};
+                      TargetWidth, static_cast<SPIRVWord>(FloatControl)) {}
 
-  SPIRVWord getTargetWidth() const { return Literals.at(0); };
+  SPIRVWord getTargetWidth() const { return Literals.at(0); }
   spv::FPRoundingMode getRoundingMode() const {
     return static_cast<spv::FPRoundingMode>(Literals.at(1));
-  };
+  }
 };
 
 class SPIRVDecorateFunctionDenormModeINTEL : public SPIRVDecorate {
@@ -646,12 +646,12 @@ class SPIRVDecorateFunctionDenormModeINTEL : public SPIRVDecorate {
                                        SPIRVWord TargetWidth,
                                        spv::FPDenormMode FloatControl)
       : SPIRVDecorate(spv::DecorationFunctionDenormModeINTEL, TheTarget,
-                      TargetWidth, static_cast<SPIRVWord>(FloatControl)){};
+                      TargetWidth, static_cast<SPIRVWord>(FloatControl)) {}
 
-  SPIRVWord getTargetWidth() const { return Literals.at(0); };
+  SPIRVWord getTargetWidth() const { return Literals.at(0); }
   spv::FPDenormMode getDenormMode() const {
     return static_cast<spv::FPDenormMode>(Literals.at(1));
-  };
+  }
 };
 
 class SPIRVDecorateFunctionFloatingPointModeINTEL : public SPIRVDecorate {
@@ -661,19 +661,19 @@ class SPIRVDecorateFunctionFloatingPointModeINTEL : public SPIRVDecorate {
                                               SPIRVWord TargetWidth,
                                               spv::FPOperationMode FloatControl)
       : SPIRVDecorate(spv::DecorationFunctionFloatingPointModeINTEL, TheTarget,
-                      TargetWidth, static_cast<SPIRVWord>(FloatControl)){};
+                      TargetWidth, static_cast<SPIRVWord>(FloatControl)) {}
 
-  SPIRVWord getTargetWidth() const { return Literals.at(0); };
+  SPIRVWord getTargetWidth() const { return Literals.at(0); }
   spv::FPOperationMode getOperationMode() const {
     return static_cast<spv::FPOperationMode>(Literals.at(1));
-  };
+  }
 };
 
 class SPIRVDecorateStallEnableINTEL : public SPIRVDecorate {
 public:
   // Complete constructor for SPIRVDecorateStallEnableINTEL
   SPIRVDecorateStallEnableINTEL(SPIRVEntry *TheTarget)
-      : SPIRVDecorate(spv::DecorationStallEnableINTEL, TheTarget){};
+      : SPIRVDecorate(spv::DecorationStallEnableINTEL, TheTarget) {}
 };
 
 class SPIRVDecorateFuseLoopsInFunctionINTEL : public SPIRVDecorate {
@@ -682,7 +682,7 @@ class SPIRVDecorateFuseLoopsInFunctionINTEL : public SPIRVDecorate {
   SPIRVDecorateFuseLoopsInFunctionINTEL(SPIRVEntry *TheTarget, SPIRVWord Depth,
                                         SPIRVWord Independent)
       : SPIRVDecorate(spv::DecorationFuseLoopsInFunctionINTEL, TheTarget, Depth,
-                      Independent){};
+                      Independent) {}
 };
 
 class SPIRVDecorateMathOpDSPModeINTEL : public SPIRVDecorate {
@@ -691,44 +691,44 @@ class SPIRVDecorateMathOpDSPModeINTEL : public SPIRVDecorate {
   SPIRVDecorateMathOpDSPModeINTEL(SPIRVEntry *TheTarget, SPIRVWord Mode,
                                   SPIRVWord Propagate)
       : SPIRVDecorate(spv::DecorationMathOpDSPModeINTEL, TheTarget, Mode,
-                      Propagate){};
+                      Propagate) {}
 };
 
 class SPIRVDecorateAliasScopeINTEL : public SPIRVDecorateId {
 public:
   // Complete constructor for SPIRVDecorateAliasScopeINTEL
   SPIRVDecorateAliasScopeINTEL(SPIRVEntry *TheTarget, SPIRVId AliasList)
-      : SPIRVDecorateId(spv::DecorationAliasScopeINTEL, TheTarget, AliasList){};
+      : SPIRVDecorateId(spv::DecorationAliasScopeINTEL, TheTarget, AliasList) {}
 };
 
 class SPIRVDecorateNoAliasINTEL : public SPIRVDecorateId {
 public:
   // Complete constructor for SPIRVDecorateNoAliasINTEL
   SPIRVDecorateNoAliasINTEL(SPIRVEntry *TheTarget, SPIRVId AliasList)
-      : SPIRVDecorateId(spv::DecorationNoAliasINTEL, TheTarget, AliasList){};
+      : SPIRVDecorateId(spv::DecorationNoAliasINTEL, TheTarget, AliasList) {}
 };
 
 class SPIRVDecorateInitiationIntervalINTEL : public SPIRVDecorate {
 public:
   // Complete constructor for SPIRVDecorateInitiationIntervalINTEL
   SPIRVDecorateInitiationIntervalINTEL(SPIRVEntry *TheTarget, SPIRVWord Cycles)
       : SPIRVDecorate(spv::DecorationInitiationIntervalINTEL, TheTarget,
-                      Cycles){};
+                      Cycles) {}
 };
 
 class SPIRVDecorateMaxConcurrencyINTEL : public SPIRVDecorate {
 public:
   // Complete constructor for SPIRVDecorateMaxConcurrencyINTEL
   SPIRVDecorateMaxConcurrencyINTEL(SPIRVEntry *TheTarget, SPIRVWord Invocations)
       : SPIRVDecorate(spv::DecorationMaxConcurrencyINTEL, TheTarget,
-                      Invocations){};
+                      Invocations) {}
 };
 
 class SPIRVDecoratePipelineEnableINTEL : public SPIRVDecorate {
 public:
   // Complete constructor for SPIRVDecoratePipelineEnableINTEL
   SPIRVDecoratePipelineEnableINTEL(SPIRVEntry *TheTarget, SPIRVWord Enable)
-      : SPIRVDecorate(spv::DecorationPipelineEnableINTEL, TheTarget, Enable){};
+      : SPIRVDecorate(spv::DecorationPipelineEnableINTEL, TheTarget, Enable) {}
 };
 
 class SPIRVDecorateHostAccessINTEL : public SPIRVDecorate {
@@ -741,7 +741,7 @@ class SPIRVDecorateHostAccessINTEL : public SPIRVDecorate {
     for (auto &I : getVec(VarName))
       Literals.push_back(I);
     WordCount += Literals.size();
-  };
+  }
 
   SPIRVWord getAccessMode() const { return Literals.front(); }
   std::string getVarName() const {
@@ -783,15 +783,15 @@ class SPIRVDecorateInitModeINTEL : public SPIRVDecorate {
   // Complete constructor for SPIRVInitModeINTEL
   SPIRVDecorateInitModeINTEL(SPIRVEntry *TheTarget, SPIRVWord Trigger)
       : SPIRVDecorate(spv::internal::DecorationInitModeINTEL, TheTarget,
-                      Trigger){};
+                      Trigger) {}
 };
 
 class SPIRVDecorateImplementInCSRINTEL : public SPIRVDecorate {
 public:
   // Complete constructor for SPIRVImplementInCSRINTEL
   SPIRVDecorateImplementInCSRINTEL(SPIRVEntry *TheTarget, SPIRVWord Value)
       : SPIRVDecorate(spv::internal::DecorationImplementInCSRINTEL, TheTarget,
-                      Value){};
+                      Value) {}
 };
 
 } // namespace SPIRV

llvm-spirv/lib/SPIRV/libSPIRV/SPIRVEntry.h

@@ -366,7 +366,7 @@ class SPIRVEntry {
   void setLinkageType(SPIRVLinkageTypeKind);
   void setModule(SPIRVModule *TheModule);
   void setName(const std::string &TheName);
-  virtual void setScope(SPIRVEntry *Scope){};
+  virtual void setScope(SPIRVEntry *Scope) {}
   void takeAnnotations(SPIRVForward *);
   void takeDecorates(SPIRVEntry *);
   void takeDecorateIds(SPIRVEntry *);

llvm-spirv/lib/SPIRV/libSPIRV/SPIRVInstruction.h

@@ -81,11 +81,11 @@ class SPIRVComponentMemorySemanticsMask {
 
 class SPIRVComponentOperands {
 public:
-  SPIRVComponentOperands(){};
+  SPIRVComponentOperands() {}
   SPIRVComponentOperands(const std::vector<SPIRVValue *> &TheOperands)
-      : Operands(TheOperands){};
+      : Operands(TheOperands) {}
   SPIRVComponentOperands(std::vector<SPIRVValue *> &&TheOperands)
-      : Operands(std::move(TheOperands)){};
+      : Operands(std::move(TheOperands)) {}
   std::vector<SPIRVValue *> getCompOperands() { return Operands; }
   std::vector<SPIRVType *> getCompOperandTypes() {
     std::vector<SPIRVType *> Tys;
@@ -2266,7 +2266,7 @@ template <Op OC> class SPIRVLifetime : public SPIRVInstruction {
       : SPIRVInstruction(3, OC, TheBB), Object(TheObject), Size(TheSize) {
     validate();
     assert(TheBB && "Invalid BB");
-  };
+  }
   // Incomplete constructor
   SPIRVLifetime()
       : SPIRVInstruction(OC), Object(SPIRVID_INVALID), Size(SPIRVWORD_MAX) {
@@ -2276,8 +2276,8 @@ template <Op OC> class SPIRVLifetime : public SPIRVInstruction {
   SPIRVCapVec getRequiredCapability() const override {
     return getVec(CapabilityKernel);
   }
-  SPIRVValue *getObject() { return getValue(Object); };
-  SPIRVWord getSize() { return Size; };
+  SPIRVValue *getObject() { return getValue(Object); }
+  SPIRVWord getSize() { return Size; }
 
 protected:
   void validate() const override {