Skip to content

[Synth] Add attribute for linear delay model and pass to approximate them from NLDM attr#10385

Open
okekayode wants to merge 2 commits intollvm:mainfrom
okekayode:dev/synth-mapping-cost
Open

[Synth] Add attribute for linear delay model and pass to approximate them from NLDM attr#10385
okekayode wants to merge 2 commits intollvm:mainfrom
okekayode:dev/synth-mapping-cost

Conversation

@okekayode
Copy link
Copy Markdown
Contributor

@okekayode okekayode commented May 5, 2026

Concerns #10382

We introduce synth.mapping_cost and switch in TechMapper from what was previously hw.techlib.info to the former. We currently consume area and intrinsic delay in TechMapper however load/phase work will be left for follow-up work.

Assisted-by: :<Codex-GPT5.5>

@okekayode okekayode force-pushed the dev/synth-mapping-cost branch from f31124c to d495248 Compare May 6, 2026 00:10
Comment thread test/Dialect/Synth/tech-mapper-error.mlir Outdated
Comment thread include/circt/Dialect/Synth/SynthAttributes.td Outdated
Comment thread include/circt/Dialect/Synth/SynthAttributes.td Outdated
@okekayode okekayode force-pushed the dev/synth-mapping-cost branch from d495248 to 9ea7451 Compare May 6, 2026 20:40
@okekayode okekayode requested a review from uenoku May 7, 2026 21:41
Comment thread include/circt/Dialect/Synth/SynthEnums.td Outdated
Comment thread lib/Dialect/Synth/Transforms/TechMapper.cpp Outdated
Comment thread lib/Dialect/Synth/Transforms/TechMapper.cpp Outdated
Comment thread include/circt/Dialect/Synth/SynthAttributes.td
Copy link
Copy Markdown
Member

@uenoku uenoku left a comment

Choose a reason for hiding this comment

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

Generally looks good to me!

@okekayode
Copy link
Copy Markdown
Contributor Author

Generally looks good to me!

Sweet! Good to squash and merge as is?

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