Skip to content

Handle mul_bits in bits_coerce_narrow transform#107

Merged
ncough merged 3 commits intoUQ-PAC:partial_evalfrom
mmcloughlin:transform-bits_coerce_narrow-mul
Sep 27, 2024
Merged

Handle mul_bits in bits_coerce_narrow transform#107
ncough merged 3 commits intoUQ-PAC:partial_evalfrom
mmcloughlin:transform-bits_coerce_narrow-mul

Conversation

@mmcloughlin
Copy link
Copy Markdown

This PR extends the bits_coerce_narrow transform to apply to mul_bits as well.

@mmcloughlin
Copy link
Copy Markdown
Author

Looks like this has changed some ASLT output and a test is failing. Please advise on how to update the test data?

@katrinafyi
Copy link
Copy Markdown
Member

See https://github.com/UQ-PAC/aslp?tab=readme-ov-file#testing. Briefly, dune runtest and then dune promote.

@mmcloughlin
Copy link
Copy Markdown
Author

See https://github.com/UQ-PAC/aslp?tab=readme-ov-file#testing. Briefly, dune runtest and then dune promote.

Thank you! Done.

@ncough ncough merged commit f0edba3 into UQ-PAC:partial_eval Sep 27, 2024
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.

3 participants