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

Pointers fixes #188

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Pointers fixes #188

merged 1 commit into from
Jul 10, 2024

Conversation

S1eGa
Copy link
Collaborator

@S1eGa S1eGa commented Jul 4, 2024

No description provided.

@S1eGa S1eGa force-pushed the pointers-fixes branch 4 times, most recently from 9a347f8 to f3386b8 Compare July 8, 2024 12:07
@S1eGa S1eGa requested a review from misonijnik July 8, 2024 14:13
@S1eGa S1eGa force-pushed the pointers-fixes branch 2 times, most recently from 073dc2d to 0844da1 Compare July 10, 2024 14:12
…dle ConstantPointer instead of ConstantExpr where it is required. Removed propagating of operations in SelectExpr (caused division on 0 for *Div operations).

[fix] Model return value of ctype_* functions as pointers.

[fix] Allow to resolve SelectExpr of ConsantExprs on all memory objects in address space.

[fix] DebugInfo may not point to any address.

[fix] Do not optimize SelectExpr for *Div operations, terminate state on call to raise function.

[refactor] Logic for writing in CType objects put in lambdas.

[test] Added tests for propagating select through division, calling to raise and for resolve of symbolic pointer represeneted as a select by a symbolic condition.

[refactor] addCType* functions moved to private methods for better readability, optimization on sifting up selects during construction of expressions moved to seperate method also for better readability.
@S1eGa S1eGa marked this pull request as ready for review July 10, 2024 18:03
@S1eGa S1eGa merged commit 78f5095 into UnitTestBot:main Jul 10, 2024
26 checks passed
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