[hipblaslt-provider] Convert hipblaslt-provider to stream-style logging

dnn-providers/hipblaslt-provider/CMakeLists.txt

@@ -114,8 +114,6 @@ target_compile_definitions(hipblaslt_plugin_private PRIVATE)
 target_include_directories(hipblaslt_plugin_private PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
 target_link_libraries(hipblaslt_plugin_private PUBLIC roc::hipblaslt hipdnn_data_sdk hipdnn_plugin_sdk hip::host)
 
-# Enable spdlog support via plugin_sdk helper function
-hipdnn_enable_spdlog(hipblaslt_plugin_private)
 target_compile_options(hipblaslt_plugin_private PRIVATE ${HIPDNN_WARNING_COMPILE_OPTIONS})
 
 add_library(hipblaslt_plugin SHARED HipblasltPluginPublic.cpp)

dnn-providers/hipblaslt-provider/HipblasltUtils.hpp

@@ -13,15 +13,15 @@
 #include <hipdnn_plugin_sdk/PluginLogging.hpp>
 #include <string>
 
-#define LOG_ON_HIPBLASLT_FAILURE(status)                                              \
-    do                                                                                \
-    {                                                                                 \
-        if(status != HIPBLAS_STATUS_SUCCESS)                                          \
-        {                                                                             \
-            HIPDNN_PLUGIN_LOG_ERROR(                                                  \
-                "hipBLASLt error occurred: {}",                                       \
-                hipblaslt_plugin::hipblaslt_utils::hipblas_status_to_string(status)); \
-        }                                                                             \
+#define LOG_ON_HIPBLASLT_FAILURE(status)                                                 \
+    do                                                                                   \
+    {                                                                                    \
+        if(status != HIPBLAS_STATUS_SUCCESS)                                             \
+        {                                                                                \
+            HIPDNN_PLUGIN_LOG_ERROR(                                                     \
+                "hipBLASLt error occurred: "                                             \
+                << hipblaslt_plugin::hipblaslt_utils::hipblas_status_to_string(status)); \
+        }                                                                                \
     } while(0)
 
 #define THROW_ON_HIPBLASLT_FAILURE(status)                                                     \

dnn-providers/hipblaslt-provider/HipdnnEnginePluginHandle.hpp

@@ -41,13 +41,13 @@ struct HipdnnEnginePluginHandle
     void storeEngineDetailsDetachedBuffer(const void* ptr,
                                           std::unique_ptr<flatbuffers::DetachedBuffer> buffer)
     {
-        HIPDNN_PLUGIN_LOG_INFO("Storing detached buffer at address: {:p}", ptr);
+        HIPDNN_PLUGIN_LOG_INFO("Storing detached buffer at address: " << ptr);
         _engineDetailsBuffers[ptr] = std::move(buffer);
     }
 
     void removeEngineDetailsDetachedBuffer(const void* ptr)
     {
-        HIPDNN_PLUGIN_LOG_INFO("Removing detached buffer at address: {:p}", ptr);
+        HIPDNN_PLUGIN_LOG_INFO("Removing detached buffer at address: " << ptr);
 
         auto it = _engineDetailsBuffers.find(ptr);
         if(it != _engineDetailsBuffers.end())
@@ -57,10 +57,11 @@ struct HipdnnEnginePluginHandle
         else
         {
             HIPDNN_PLUGIN_LOG_WARN(
-                "No detached buffer found at address: {:p}. Could not remove engine "
-                "details. Ensure you "
-                "are using the same hipdnn handle you used for engine details creation",
-                ptr);
+                "No detached buffer found at address: "
+                << ptr
+                << ". Could not remove engine "
+                   "details. Ensure you "
+                   "are using the same hipdnn handle you used for engine details creation");
         }
     }
 

dnn-providers/hipblaslt-provider/engines/plans/HipblasltMatmulPlanBuilder.cpp

@@ -119,7 +119,7 @@ void HipblasltMatmulPlanBuilder::buildPlan(
     validateGraphConfiguration(opGraph);
 
     const auto& nodeWrapper = opGraph.getNodeWrapper(0);
-    HIPDNN_PLUGIN_LOG_INFO("Building matmul plan for node: {}", nodeWrapper.name());
+    HIPDNN_PLUGIN_LOG_INFO("Building matmul plan for node: " << nodeWrapper.name());
 
     const auto& attr = nodeWrapper.attributesAs<hipdnn_data_sdk::data_objects::MatmulAttributes>();
     MatmulParams params(attr, opGraph.getTensorMap());

dnn-providers/hipblaslt-provider/integration_tests/CMakeLists.txt

@@ -25,9 +25,6 @@ target_link_libraries(
     hipblaslt_plugin_integration_tests PUBLIC GTest::gtest hip::host hipdnn_test_sdk hipdnn_plugin_sdk Threads::Threads
 )
 
-# Enable spdlog support for fmt-style logging
-hipdnn_enable_spdlog(hipblaslt_plugin_integration_tests)
-
 # Ensure that the hipblaslt_plugin is built before the hipblaslt_plugin_integration_tests, as the
 # integration tests depend on the hipBLASLt Provider Plugin
 add_dependencies(hipblaslt_plugin_integration_tests hipblaslt_plugin)

dnn-providers/hipblaslt-provider/integration_tests/IntegrationGraphVerificationHarness.hpp

@@ -89,7 +89,7 @@ class IntegrationGraphVerificationHarness : public ::testing::TestWithParam<Test
             << "At least one output tensor id must be specified for "
                "validation.";
 
-        HIPDNN_PLUGIN_LOG_INFO("Validating {} output tensors", outputTensorIds.size());
+        HIPDNN_PLUGIN_LOG_INFO("Validating " << outputTensorIds.size() << " output tensors");
 
         // Lazily register validators after graph execution since tensor Ids and types may be inferred during graph finalization
         for(const auto& registerValidator : _deferredValidators)

dnn-providers/hipblaslt-provider/integration_tests/main.cpp

@@ -4,7 +4,6 @@ SPDX-License-Identifier: MIT
 */
 
 #include <gtest/gtest.h>
-#include <spdlog/spdlog.h>
 
 #include <hipdnn_frontend.hpp>
 #include <hipdnn_test_sdk/utilities/HipErrorHandler.hpp>
@@ -19,7 +18,5 @@ int main(int argc, char** argv)
     testing::TestEventListeners& listeners = testing::UnitTest::GetInstance()->listeners();
     listeners.Append(new hipdnn_test_sdk::utilities::HipErrorHandler);
 
-    auto result = RUN_ALL_TESTS();
-    spdlog::shutdown();
-    return result;
+    return RUN_ALL_TESTS();
 }

dnn-providers/hipblaslt-provider/tests/main.cpp

@@ -4,7 +4,6 @@ SPDX-License-Identifier: MIT
 */
 
 #include <gtest/gtest.h>
-#include <spdlog/spdlog.h>
 
 #include <hipdnn_plugin_sdk/PluginLogging.hpp>
 #include <hipdnn_test_sdk/utilities/HipErrorHandler.hpp>
@@ -20,7 +19,5 @@ int main(int argc, char** argv)
     testing::TestEventListeners& listeners = testing::UnitTest::GetInstance()->listeners();
     listeners.Append(new hipdnn_test_sdk::utilities::HipErrorHandler);
 
-    auto result = RUN_ALL_TESTS();
-    spdlog::shutdown();
-    return result;
+    return RUN_ALL_TESTS();
 }