Clean up code

maxnick · Jan 15, 2025 · 158285f · 158285f
1 parent 0382ce8
commit 158285f
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 3 deletions.
diff --git a/src/plugins/intel_cpu/src/utils/debug_caps_config.h b/src/plugins/intel_cpu/src/utils/debug_caps_config.h
@@ -47,7 +47,7 @@ class DebugCapsConfig {
     std::unordered_map<FILTER, std::string, EnumClassHash> blobDumpFilters;
     std::string summaryPerf = "";
     std::string memoryStatisticsDumpPath;
-    unsigned memoryStatisticsDumpLevel = 0;  // 0 - no, 1 - only the main graph, 2 - all subgraphs
+    unsigned memoryStatisticsDumpLevel = 0;  // 0 - don't print statistics, 1 - print statistics
 
     struct TransformationFilter {
         enum Type : uint8_t { PreLpt = 0, Lpt, PostLpt, Snippets, Specific, NumOfTypes };

diff --git a/src/plugins/intel_cpu/src/utils/memory_stats_dump.cpp b/src/plugins/intel_cpu/src/utils/memory_stats_dump.cpp
@@ -122,8 +122,8 @@ void dumpMemoryStats(const DebugCapsConfig& conf,
         return;
     }
 
-    std::filesystem::path csv_path = conf.memoryStatisticsDumpPath;
-    if (!csv_path.empty()) {
+    if (!conf.memoryStatisticsDumpPath.empty()) {
+        std::filesystem::path csv_path = conf.memoryStatisticsDumpPath;
         // check the last symbols in the file path are csv
         if (csv_path.extension() != ".csv") {
             OPENVINO_THROW("Unsupported memory stats file extension. Should be '*.csv'. Got ", csv_path.filename());

diff --git a/src/plugins/intel_cpu/src/weights_cache.cpp b/src/plugins/intel_cpu/src/weights_cache.cpp
@@ -89,6 +89,7 @@ const WeightsSharing::Ptr& SocketsWeights::operator[](int socket_id) const {
     return found->second;
 }
 
+#ifdef CPU_DEBUG_CAPS
 WeightsSharing::Statistics WeightsSharing::dumpStatistics() const {
     Statistics retVal = {0, 0};
 
@@ -115,5 +116,6 @@ std::vector<std::pair<int, WeightsSharing::Statistics>> SocketsWeights::dumpStat
 
     return retVal;
 }
+#endif // CPU_DEBUG_CAPS
 }  // namespace intel_cpu
 }  // namespace ov
diff --git a/src/plugins/intel_cpu/src/weights_cache.hpp b/src/plugins/intel_cpu/src/weights_cache.hpp
@@ -40,10 +40,12 @@ class WeightsSharing {
     };
 
 public:
+#ifdef CPU_DEBUG_CAPS
     struct Statistics {
         size_t total_size;  // bytes
         size_t total_memory_objects;
     };
+#endif  // CPU_DEBUG_CAPS
 
     typedef std::shared_ptr<WeightsSharing> Ptr;
 
@@ -67,7 +69,9 @@ class WeightsSharing {
 
     SharedMemory::Ptr get(const std::string& key) const;
 
+#ifdef CPU_DEBUG_CAPS
     Statistics dumpStatistics() const;
+#endif // CPU_DEBUG_CAPS
 
 protected:
     mutable std::mutex guard;
@@ -86,7 +90,9 @@ class SocketsWeights {
     WeightsSharing::Ptr& operator[](int i);
     const WeightsSharing::Ptr& operator[](int i) const;
 
+#ifdef CPU_DEBUG_CAPS
     std::vector<std::pair<int, WeightsSharing::Statistics>> dumpStatistics() const;
+#endif // CPU_DEBUG_CAPS
 
 private:
     std::map<int, WeightsSharing::Ptr> _cache_map;