Skip to content

Conversation

@uenoku
Copy link
Member

@uenoku uenoku commented Jul 18, 2025

This modifies circt-synth to make it more practical.

  • Remove verification code from synthesis parts. ExtarctTestCode is currently used, but eventually it must be replaced with something cleaner.
  • Add an option to disable wordsToBits.

@uenoku uenoku requested a review from Copilot July 18, 2025 06:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modifies circt-synth to make it more practical for synthesis by adding options to handle verification code and bit-blasting optimizations.

  • Adds verification code extraction to move test code away from design modules for faster synthesis
  • Introduces an option to disable WordsToBits transformation for better scalability with large designs
  • Updates build dependencies to support the new verification code extraction functionality

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tools/circt-synth/circt-synth.cpp Adds SV header include and ExtractTestCode pass to synthesis pipeline
tools/circt-synth/CMakeLists.txt Adds CIRCTSVTransforms dependency for ExtractTestCode pass
test/circt-synth/basic.mlir Adds test case for the new disable-word-to-bits option
lib/Synthesis/SynthesisPipeline.cpp Makes LowerWordToBits pass conditional based on options
include/circt/Synthesis/SynthesisPipeline.h Defines disableWordToBits option in pipeline options

@uenoku uenoku force-pushed the dev/hidetou/tweak-synth branch 3 times, most recently from 1b2ef8f to 31582ab Compare July 18, 2025 06:29
@uenoku uenoku force-pushed the dev/hidetou/tweak-synth branch from 31582ab to b3e1c8b Compare July 18, 2025 07:05
Copy link
Member

@maerhart maerhart left a comment

Choose a reason for hiding this comment

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

LGTM

@uenoku uenoku merged commit ba24efc into llvm:main Jul 18, 2025
7 checks passed
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