google / google/xls

Reconsider parameters used for delay model regression curve fit for multiplies (umul, smul)

Open
#1,022 0 comments 0 reactions 0 assignees View on GitHub
delay model
Dominant language
C++
Stars
1.9k
Forks
283
Avg merge
2d 10h
Merged PRs (30d)
135

Description

Currently, the sky130 delay model uses just the bit width of the first operand when estimating a particular multiplication's hardware delay.

Other delay models use just the *result* bit width for mult delay estimation.

Neither is sufficient since they are independent and both contributing factors. In fact the second operand's bit width also contributes, although perhaps the two operand widths can be combined into a single expression, so that the regression curve fitting has two expressions (combination of operand widths, and result width).

Consider:
u8 * u8 --> u8
u8 * u8 --> u16
u8 * u16 --> u16

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.