Add override for virtual functions which are overriden (#1261)

* Enable warnings for non-virtual destructors and missing overrides; add override and virtual destructors to remove warnings

* .git-blame-ignore-revs

Author: leekillough

.git-blame-ignore-revs

@@ -1,2 +1,5 @@
+# Enable warnings for non-virtual destructors and missing overrides; add override and virtual destructors to remove warnings
+f150d6f7db2d71f4a77720ee7e7a7385a57bf540 
+
 # Initial cmake-format fixes
 1c149870b806f5412c34c918cea96caa14720f3c

configure.ac

@@ -62,7 +62,7 @@ esac
 
 SST_CHECK_PICKY
 AS_IF([test "x$use_picky" = "xyes"],
-      [WARNFLAGS="-Wall -Wextra -Wvla"],
+      [WARNFLAGS="-Wall -Wextra -Wvla -Wnon-virtual-dtor -Wsuggest-override"],
       [WARNFLAGS=""])
 CFLAGS="$CFLAGS $WARNFLAGS"
 CXXFLAGS="$CXXFLAGS $WARNFLAGS"

src/sst/core/impl/timevortex/timeVortexPQ.cc

@@ -153,7 +153,10 @@ class TimeVortexPQ : public TimeVortexPQBase<false>
     TimeVortexPQ() : TimeVortexPQBase<false>() {} // For serialization only
     ~TimeVortexPQ() {}
 
-    void serialize_order(SST::Core::Serialization::serializer& ser) { TimeVortexPQBase<false>::serialize_order(ser); }
+    void serialize_order(SST::Core::Serialization::serializer& ser) override
+    {
+        TimeVortexPQBase<false>::serialize_order(ser);
+    }
 
     SST_ELI_EXPORT(TimeVortexPQ)
 };
@@ -175,7 +178,10 @@ class TimeVortexPQ_ts : public TimeVortexPQBase<true>
     TimeVortexPQ_ts() : TimeVortexPQBase<true>() {} // For serialization only
     ~TimeVortexPQ_ts() {}
 
-    void serialize_order(SST::Core::Serialization::serializer& ser) { TimeVortexPQBase<true>::serialize_order(ser); }
+    void serialize_order(SST::Core::Serialization::serializer& ser) override
+    {
+        TimeVortexPQBase<true>::serialize_order(ser);
+    }
 
     SST_ELI_EXPORT(TimeVortexPQ_ts)
 };

src/sst/core/interfaces/stdMem.h

@@ -1538,7 +1538,10 @@ class StandardMem : public SubComponent
     /**
      * Serialization function
      */
-    virtual void serialize_order(SST::Core::Serialization::serializer& ser) { SST::SubComponent::serialize_order(ser); }
+    void serialize_order(SST::Core::Serialization::serializer& ser) override
+    {
+        SST::SubComponent::serialize_order(ser);
+    }
 };
 
 } // namespace SST::Interfaces

src/sst/core/testElements/coreTest_ClockerComponent.h

@@ -47,8 +47,8 @@ class coreTestClockerComponent : public SST::Component
     )
 
     coreTestClockerComponent(SST::ComponentId_t id, SST::Params& params);
-    void setup() {}
-    void finish() {}
+    void setup() override {}
+    void finish() override {}
 
 private:
     coreTestClockerComponent();                                                    // for serialization only

src/sst/core/testElements/coreTest_DistribComponent.h

@@ -57,8 +57,8 @@ class coreTestDistribComponent : public SST::Component
     )
 
     coreTestDistribComponent(SST::ComponentId_t id, SST::Params& params);
-    void finish();
-    void setup() {}
+    void finish() override;
+    void setup() override {}
 
 private:
     coreTestDistribComponent();                                                    // for serialization only

src/sst/core/testElements/coreTest_Links.h

@@ -53,8 +53,8 @@ class coreTestLinks : public SST::Component
     coreTestLinks(SST::ComponentId_t id, SST::Params& params);
     ~coreTestLinks() = default;
 
-    void setup() {}
-    void finish() {}
+    void setup() override {}
+    void finish() override {}
 
 private:
     int my_id;

src/sst/core/testElements/coreTest_MessageGeneratorComponent.h

@@ -50,8 +50,8 @@ class coreTestMessageGeneratorComponent : public SST::Component
     )
 
     coreTestMessageGeneratorComponent(SST::ComponentId_t id, SST::Params& params);
-    void setup() {}
-    void finish()
+    void setup() override {}
+    void finish() override
     {
         fprintf(stdout, "Component completed at: %" PRIu64 " milliseconds\n", (uint64_t)getCurrentSimTimeMilli());
     }

src/sst/core/testElements/coreTest_ParamComponent.h

@@ -58,8 +58,8 @@ class coreTestParamComponent : public SST::Component
 
     coreTestParamComponent(SST::ComponentId_t id, SST::Params& params);
     ~coreTestParamComponent() {}
-    void setup() {}
-    void finish() {}
+    void setup() override {}
+    void finish() override {}
 
 private:
     coreTestParamComponent();                                                  // for serialization only

src/sst/core/testElements/coreTest_PerfComponent.h

@@ -99,8 +99,8 @@ class coreTestPerfComponent : public coreTestPerfComponentBase2
     coreTestPerfComponent(SST::ComponentId_t id, SST::Params& params);
     ~coreTestPerfComponent();
 
-    void setup() {}
-    void finish() { printf("Perf Test Component Finished.\n"); }
+    void setup() override {}
+    void finish() override { printf("Perf Test Component Finished.\n"); }
 
 private:
     coreTestPerfComponent();                                                 // for serialization only