Address review comments

Author: aobolensk

src/common/snippets/docs/debug_capabilities/linear_ir_passes_serialization.md

@@ -17,19 +17,25 @@ Examples:
     OV_SNIPPETS_DUMP_LIR="passes=all dir=path/dumpdir formats=control_flow" binary ...
     OV_SNIPPETS_DUMP_LIR="passes=FuseLoops,InsertLoops,InsertLoadStore formats=data_flow" binary ...
     OV_SNIPPETS_DUMP_LIR="passes=final formats=control_flow name_modifier=subgraph_name" binary ...
+    OV_SNIPPETS_DUMP_LIR="passes=all dir=path/dumpdir name_modifier=branchA" binary ...
 ```
 
+Dumped files have below names:
+ - Regular passes: `lir_<index>_<pass>_(control_flow|data_flow)_(in|out).xml` (creates both input and output files)
+ - Final dump: `lir_<index>_Final_(control_flow|data_flow).xml` (creates single file only)
+
+When `name_modifier` is provided, it is prepended to file names as a prefix:
+ - `name_modifier=subgraph_name` prepends the Snippets Subgraph friendly name where available (e.g., final dump).
+ - any other non-empty value prepends that literal value (e.g., `branchA_lir_...`).
+
 Option names are case insensitive, the following options are supported:
  - `passes` : Dump LIR around the passes if passes name are specified.
  It support multiple comma separated pass names. The names are case insensitive.
  This option is a must have, should not be omitted.
- Special values: 'all' - dump all passes (includes 'final'), 'final' - dump final LIR snapshot right before code generation.
+ Special values: 'all' - dump all passes (includes 'final'), 'final' - dump final LIR snapshot right before code generation (single file, no _in/_out suffix).
  - `dir` : Path to dumped LIR files.
  If omitted, it defaults to snippets_LIR_dump.
  If specified path doesn't exist, the directory will be created automatically.
  - `formats` : Support values are control_flow, data_flow and all.
  If omitted, it defaults to control_flow.
- - `name_modifier` : Optional naming modifier for dump files. Supported value is `subgraph_name`, which prepends the Snippets Subgraph friendly name to dumped files. If omitted, no name modification is performed. Note: when using `subgraph_name`, characters '/' and ':' in the subgraph name are replaced with '_' for filesystem compatibility.
-
-Notes:
-- File names follow the same `OV_SNIPPETS_DUMP_LIR` convention: `lir_<index>_<pass>_(control_flow|data_flow)_(in|out).xml`. When `name_modifier=subgraph_name` is set, it is prepended to the file name.
+ - `name_modifier` : Optional file-name prefix. Special value `subgraph_name` prepends the Snippets Subgraph friendly name to dumped files (where available). Any other non-empty value is used as a literal prefix. If omitted, no name modification is performed. Note: when using `subgraph_name`, characters '/' and ':' in the subgraph name are replaced with '_' for filesystem compatibility.

src/common/snippets/include/snippets/utils/debug_caps_config.hpp

@@ -51,7 +51,6 @@ class DebugCapsConfig {
         std::string dir = "snippets_LIR_dump";
         LIRFormatFilter format = {1 << LIRFormatFilter::controlFlow};
         std::vector<std::string> passes;
-        bool use_subgraph_name = false;
         std::string name_modifier;
 
         std::vector<PropertySetterPtr> getPropertySetters() override {
@@ -63,9 +62,11 @@ class DebugCapsConfig {
                         "indicate dump LIRs around the passes. Support multiple passes with comma separated and case "
                         "insensitive. Special values: 'all' - dump all passes (includes 'final'), 'final' - dump final "
                         "LIR")),
-                    PropertySetterPtr(new StringPropertySetter("name_modifier",
-                                                               name_modifier,
-                                                               "optional name modifier, supported: subgraph_name"))};
+                    PropertySetterPtr(new StringPropertySetter(
+                        "name_modifier",
+                        name_modifier,
+                        "optional file-name prefix; special value 'subgraph_name' uses the Subgraph friendly name; any "
+                        "other non-empty value is used as a literal prefix"))};
         }
     } dumpLIR;
 

src/common/snippets/include/snippets/utils/linear_ir_pass_dumper.hpp

@@ -18,25 +18,40 @@ namespace ov::snippets {
 
 class LIRPassDump {
 public:
-    explicit LIRPassDump(const lowered::LinearIR& linear_ir, std::string pass_name, std::string name_prefix = "")
+    enum class DumpMode : uint8_t { Both, SingleDump };
+
+    explicit LIRPassDump(const lowered::LinearIR& linear_ir,
+                         std::string pass_name,
+                         std::string name_prefix = "",
+                         DumpMode mode = DumpMode::Both)
         : linear_ir(linear_ir),
           pass_name(std::move(pass_name)),
           name_prefix(std::move(name_prefix)),
+          dump_mode(mode),
           debug_config(*linear_ir.get_config().debug_config) {
-        dump("_in");
+        if (dump_mode == DumpMode::Both) {
+            dump("_in");
+        } else {
+            dump("");
+        }
     }
     ~LIRPassDump() {
-        dump("_out");
+        if (dump_mode == DumpMode::Both) {
+            dump("_out");
+        }
     }
 
 private:
     void dump(const std::string&& postfix) const {
         static int num = 0;  // just to keep dumped IRs ordered in filesystem
         auto pathAndName = debug_config.dumpLIR.dir + "/";
-        const bool use_subgraph_prefix =
-            ov::util::to_lower(debug_config.dumpLIR.name_modifier) == std::string("subgraph_name");
-        if (use_subgraph_prefix && !name_prefix.empty()) {
-            pathAndName += name_prefix + "_";
+        const auto nm_lower = ov::util::to_lower(debug_config.dumpLIR.name_modifier);
+        if (nm_lower == std::string("subgraph_name")) {
+            if (!name_prefix.empty()) {
+                pathAndName += name_prefix + "_";
+            }
+        } else if (!debug_config.dumpLIR.name_modifier.empty()) {
+            pathAndName += debug_config.dumpLIR.name_modifier + "_";
         }
         pathAndName += "lir_";
 
@@ -60,6 +75,7 @@ class LIRPassDump {
     const lowered::LinearIR& linear_ir;
     const std::string pass_name;
     const std::string name_prefix;
+    const DumpMode dump_mode;
     const DebugCapsConfig& debug_config;
 };
 

src/common/snippets/src/op/subgraph.cpp

@@ -637,7 +637,7 @@ snippets::Schedule Subgraph::generate(const void* compile_params) const {
             std::replace(name_prefix.begin(), name_prefix.end(), '/', '_');
             std::replace(name_prefix.begin(), name_prefix.end(), ':', '_');
         }
-        LIRPassDump final_dump(*linear_ir, std::string("Final"), name_prefix);
+        LIRPassDump final_dump(*linear_ir, std::string("Final"), name_prefix, LIRPassDump::DumpMode::SingleDump);
     }
 #endif