Skip to content

Conversation

uenoku
Copy link
Member

@uenoku uenoku commented Jul 2, 2025

This patch improves the AIG longest path analysis pass with:

  • Detailed timing level statistics showing delay distribution
  • Critical path details for top K fanout points
  • Path history with intermediate debug points
  • JSON output format option for machine-readable results
  • Improved documentation and command-line options

The enhanced analysis provides more comprehensive timing information for debugging and optimizing AIG circuits, with both human-readable and machine-parseable output formats. Human-readable output is align with ABC's print_level output to make it easy to compare results with ABC.

…nd JSON output

This patch improves the AIG longest path analysis pass with:
- Detailed timing level statistics showing delay distribution
- Critical path details for top K fanout points
- Path history with intermediate debug points
- JSON output format option for machine-readable results
- Improved documentation and command-line options

The enhanced analysis provides more comprehensive timing information
for debugging and optimizing AIG circuits, with both human-readable
and machine-parseable output formats.
@uenoku uenoku requested a review from fabianschuiki July 2, 2025 21:38
@fabianschuiki fabianschuiki changed the title [AIG] Enhance longest path analysis with detailed timing statistics andJSON output [AIG] Enhance longest path analysis with detailed timing statistics and JSON output Jul 3, 2025
Copy link
Contributor

@fabianschuiki fabianschuiki left a comment

Choose a reason for hiding this comment

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

LGTM! Really cool to get such detailed output and timing info 🥳

Comment on lines +41 to +42
Option<"emitJSON", "emit-json", "bool", "false",
"Output analysis results in JSON format">
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool!

Comment on lines +1 to +2
// RUN: circt-synth %s -output-longest-path=- -top counter | FileCheck %s
// RUN: circt-synth %s -output-longest-path=- -top counter -output-longest-path-json | FileCheck %s --check-prefix JSON
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice to have - available.

@uenoku uenoku merged commit 02eff06 into main Jul 3, 2025
7 checks passed
@uenoku uenoku deleted the dev/hidetou/aig-report branch July 3, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants