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

Merge upstream #3

Merged
merged 19 commits into from
Nov 19, 2022
Merged

Merge upstream #3

merged 19 commits into from
Nov 19, 2022

Conversation

julihoh
Copy link
Collaborator

@julihoh julihoh commented Nov 19, 2022

aurelf and others added 19 commits May 28, 2022 15:59
This commit needs to be reverted once a proper fix is in place.
This is just the output of "clang-format -style=llvm -dump-config".
Clang now uses the new pass manager for the optimization pipeline, so we
have to do the same to make Clang use our pass. Moreover, FileCheck now
complains if a configured prefix doesn't appear in the checked file; added
"ANY" in three tests where it was missing. Finally, printing
arbitrary-precision integers in QSYM needed some changes.
We should really automate this...
Z3 doesn't always output model constants in the same order; make sure
that our tests don't depend on it.
This commit adds the option to mark symbolic input by calling
symcc_make_symbolic from the program under test.

The refactoring that was required to add the new feature has had the
pleasant side effect that the QSYM backend now doesn't require the
entire input upfront anymore, making it much more convenient to feed
symbolic data through stdin.
No need for "push": the "pull_request" event already triggers when new
commits are pushed to the PR branch, and we expect all changes to go
through a PR.
@julihoh julihoh requested a review from domenukk November 19, 2022 13:52
@julihoh
Copy link
Collaborator Author

julihoh commented Nov 19, 2022

I had to do some manual merging. Semantic differences were in Cargo.toml/Cargo.lock. I chose the upstream changes, which I believe are fine.

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.

5 participants