[Hipcc] Mark ~HipBinBase as virutal

For hipBinNVPtr_ and hipBinAMDPtr_ members: the destructor of the base
class was not marked as virtual, but the destructor of the derived
classes are. When we delete the object we do it through a
pointer to the base class. So the base class destructor is called but
not the one of the derived classes. This results in strange memory
behaviour detected by ASAN.

Solves SWDEV-516418

Author: jmmartinez

amd/hipcc/src/hipBin_amd.h

@@ -51,7 +51,7 @@ class HipBinAmd : public HipBinBase {
 
  public:
   HipBinAmd();
-  virtual ~HipBinAmd() = default;
+  ~HipBinAmd() override = default;
   virtual bool detectPlatform();
   virtual void constructCompilerPath();
   virtual const string& getCompilerPath() const;

amd/hipcc/src/hipBin_base.h

@@ -203,6 +203,7 @@ enum HipBinCommand {
 class HipBinBase {
  public:
   HipBinBase();
+  virtual ~HipBinBase() = default;
   // Interface functions
   virtual void constructCompilerPath() = 0;
   virtual void printFull() = 0;

amd/hipcc/src/hipBin_nvidia.h

@@ -37,7 +37,7 @@ class HipBinNvidia : public HipBinBase {
 
  public:
   HipBinNvidia();
-  virtual ~HipBinNvidia() = default;
+  ~HipBinNvidia() override = default;
   virtual bool detectPlatform();
   virtual void constructCompilerPath();
   virtual const string& getCompilerPath() const;