[OpenMP] Missing virtual destructor in KMPAffinity

[grokos]

Summary

Class KMPAffinity has virtual functions, therefore it needs a virtual destructor too. If the virtual destructor is missing, the compiler emits a warning, compiling with -Werror makes the compilation fail.

[jlpeyton]

LGTM

runtime/src/kmp.h

public:
         // Set this affinity mask to the current system affinity
         virtual int get_system_affinity(bool abort_on_error) { return -1; }
         // Only 1 DWORD in the mask should have any procs set.
         // Return the appropriate index, or -1 for an invalid mask.
         virtual int get_proc_group() const { return -1; }
     };
     void* operator new(size_t n);
     void operator delete(void* p);
+    // Need virtual destructor
+    virtual ~KMPAffinity() = default;
     // Determine if affinity is capable
     virtual void determine_capable(const char* env_var) {}
     // Bind the current thread to os proc
     virtual void bind_thread(int proc) {}
     // Factory functions to allocate/deallocate a mask
     virtual Mask* allocate_mask() { return nullptr; }
     virtual void deallocate_mask(Mask* m) { }
     virtual Mask* allocate_mask_array(int num) { return nullptr; }