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

Author: leekillough

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

@@ -1523,7 +1523,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

src/sst/core/testElements/coreTest_RNGComponent.h

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