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

Ian/sleigh support rebased #607

Merged
merged 230 commits into from
Jul 28, 2022
Merged

Ian/sleigh support rebased #607

merged 230 commits into from
Jul 28, 2022

Conversation

2over12
Copy link
Contributor

@2over12 2over12 commented Jun 28, 2022

This pull request implements lifting to llvm via sleigh semantics. The PR adds the lifting-bits sleigh repo as a dependency.

@2over12 2over12 assigned lkorenc and unassigned lkorenc Jun 29, 2022
@2over12 2over12 requested a review from lkorenc June 29, 2022 12:09
func1_state->aflag.pf = test_runner::random_boolean_flag(this->rbe);
func1_state->aflag.sf = test_runner::random_boolean_flag(this->rbe);
func1_state->aflag.zf = test_runner::random_boolean_flag(this->rbe);
X86State func1_state = {};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

= {} is not needed, default ctor should be called even if you just do T val;

VarnodeData target_vnode) {
if (auto res = this->LiftNormalRegister(bldr, reg_name)) {
return *res;
} else {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else after return.

@2over12 2over12 merged commit eef338d into master Jul 28, 2022
@ekilmer ekilmer deleted the ian/sleigh-support-rebased branch March 25, 2023 19:55
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.

6 participants