Skip to content

Conversation

aobolensk
Copy link
Contributor

Details:

Add support for dumping final Linear IR right before code generation using the existing OV_SNIPPETS_DUMP_LIR debug capability.

  • Extend OV_SNIPPETS_DUMP_LIR to support "final" pass name for capturing the final state of Linear IR before code generation
  • Add name_modifier option to include subgraph friendly names in dump filenames
  • Sanitize subgraph names by replacing '/' and ':' with '_'
  • Update LIRPassDump to support optional name prefix for better file identification when multiple subgraphs are present

Usage:

OV_SNIPPETS_DUMP_LIR="passes=final name_modifier=subgraph_name" binary

Tickets:

  • N/A

@aobolensk aobolensk requested review from a team as code owners September 9, 2025 12:16
@aobolensk aobolensk requested review from zKulesza and removed request for a team September 9, 2025 12:16
@github-actions github-actions bot added category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation labels Sep 9, 2025
@aobolensk aobolensk requested a review from v-Golubev September 10, 2025 11:58
@aobolensk aobolensk force-pushed the snippets=debug-caps-final-lir-dump branch 2 times, most recently from c6001e0 to bba02b1 Compare September 10, 2025 17:59
@aobolensk aobolensk force-pushed the snippets=debug-caps-final-lir-dump branch from ecbacf8 to 52abb9b Compare September 15, 2025 05:20
Add support for dumping final Linear IR right before code generation using the existing OV_SNIPPETS_DUMP_LIR debug capability.

 - Extend OV_SNIPPETS_DUMP_LIR to support "final" pass name for capturing the final state of Linear IR before code generation
 - Add name_modifier option to include subgraph friendly names in dump filenames
 - Sanitize subgraph names by replacing '/' and ':' with '_' for filesystem compatibility
 - Update LIRPassDump to support optional name prefix for better file identification when multiple subgraphs are present

Usage:
```bash
OV_SNIPPETS_DUMP_LIR="passes=final name_modifier=subgraph_name" binary
```
Copy link
Contributor

@v-Golubev v-Golubev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job 👍

@v-Golubev v-Golubev enabled auto-merge September 16, 2025 10:16
@v-Golubev v-Golubev added this pull request to the merge queue Sep 16, 2025
auto-merge was automatically disabled September 16, 2025 21:00

Pull Request is not mergeable

Merged via the queue into openvinotoolkit:master with commit a110520 Sep 16, 2025
202 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants