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

Side-effects of memory / register loads #25

Open
katrinafyi opened this issue Mar 21, 2023 · 0 comments
Open

Side-effects of memory / register loads #25

katrinafyi opened this issue Mar 21, 2023 · 0 comments

Comments

@katrinafyi
Copy link
Member

It is unclear whether memory and/or register loads should be considered side-effecting. This affects downstream uses and transformations in significant ways. Most notably, side-effecting operations should be excluded from unused code elimination.

Currently, these are treated as not side-effecting, but this causes inaccuracies in the later Alive2 evaluation. Since the loads may be—but are not always—removed, this can create mismatches in the translation validation where it sees an unmatched load() intrinsic on only one side.

To make this more consistent, we would need to define the semantics and implement them across the toolchain. Additionally, it would be nice to verify that other lifters under test are similar in this way.

This issue is to record some thoughts from #21.

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

No branches or pull requests

1 participant