Fix warnings

Signed-off-by: Alexander Batashev <[email protected]>

Author: Alexander Batashev

sycl/include/CL/sycl/detail/image_impl.hpp

@@ -192,7 +192,7 @@ template <int Dimensions> class image_impl final : public SYCLMemObjT {
   size_t get_count() const { return MRange.size(); }
 
   void *allocateMem(ContextImplPtr Context, bool InitFromUserData,
-                    void *HostPtr, RT::PiEvent &OutEventToWait);
+                    void *HostPtr, RT::PiEvent &OutEventToWait) override;
 
   MemObjType getType() const override { return MemObjType::IMAGE; }
 
@@ -288,10 +288,6 @@ template <int Dimensions> class image_impl final : public SYCLMemObjT {
   size_t MRowPitch = 0;
   size_t MSlicePitch = 0;
 };
-
-template class image_impl<1>;
-template class image_impl<2>;
-template class image_impl<3>;
 } // namespace detail
 } // namespace sycl
 } // namespace cl

sycl/include/CL/sycl/detail/sycl_mem_obj_i.hpp

@@ -28,6 +28,8 @@ using ContextImplPtr = std::shared_ptr<detail::context_impl>;
 // objects.
 class SYCLMemObjI {
 public:
+  virtual ~SYCLMemObjI() = default;
+
   enum MemObjType { BUFFER, IMAGE };
 
   virtual MemObjType getType() const = 0;

sycl/source/detail/image_impl.cpp

@@ -258,8 +258,10 @@ image_impl<Dimensions>::image_impl(cl_mem MemObject, const context &SyclContext,
   switch (Dimensions) {
   case 3:
     getImageInfo(PI_IMAGE_INFO_DEPTH, MRange[2]);
+    // fall through
   case 2:
     getImageInfo(PI_IMAGE_INFO_HEIGHT, MRange[1]);
+    // fall through
   case 1:
     getImageInfo(PI_IMAGE_INFO_WIDTH, MRange[0]);
   }
@@ -401,6 +403,11 @@ bool image_impl<Dimensions>::checkImageFormat(
 
   return true;
 }
+
+template class image_impl<1>;
+template class image_impl<2>;
+template class image_impl<3>;
+
 } // namespace detail
 } // namespace sycl
 } // namespace cl