Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace remaining usage of old arithmetic overflow operations #1483

Open
zhassan-aws opened this issue Aug 9, 2022 · 0 comments
Open

Replace remaining usage of old arithmetic overflow operations #1483

zhassan-aws opened this issue Aug 9, 2022 · 0 comments
Labels
[C] Bug This is a bug. Something isn't working.

Comments

@zhassan-aws
Copy link
Contributor

In #1426, new methods for arithmetic overflow operations were added that rely on CBMC's new, more efficient operations: add_overflow_result, sub_overflow_result, and mul_overflow_result. There are still a few usages of the old methods, e.g. add_overflow in kani_compiler::codegen_cprover_gotoc::codegen::intrinsic::GotocCtx::codegen_offset. We should switch all the usages to the new methods, and delete the old methods.

@zhassan-aws zhassan-aws added the [C] Bug This is a bug. Something isn't working. label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Bug This is a bug. Something isn't working.
Projects
None yet
Development

No branches or pull requests

1 participant