Skip to content

feat(dataflow): known bits analysis - #1374

Draft
axelcool1234 wants to merge 6 commits into
opencompl:mainfrom
axelcool1234:known-bits
Draft

axelcool1234 wants to merge 6 commits into
opencompl:mainfrom
axelcool1234:known-bits

Conversation

@axelcool1234

Copy link
Copy Markdown
Collaborator

No description provided.

@axelcool1234
axelcool1234 marked this pull request as draft September 4, 2026 08:08
@axelcool1234
axelcool1234 force-pushed the known-bits branch 7 times, most recently from 03ac6ba to bc01901 Compare September 10, 2026 18:42
@regehr

regehr commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

one thing you should add here is an invariant on the known bits struct that no bit is known both zero and one.

another thing you should do is to think carefully about top. here you have an explicit representation for it, but there's another representation, which is that no bits are known. it's a mistake to be able to represent the same value two different ways. you should either remove the explicit top or else use another invariant on the struct to enforce that at least one bit is always known.

@axelcool1234
axelcool1234 force-pushed the known-bits branch 3 times, most recently from 51c638c to d523ccc Compare September 12, 2026 19:14
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