Skip to content

Conversation

@pgoodman
Copy link
Contributor

@pgoodman pgoodman commented Jan 7, 2025

No description provided.

[submodule "vendor/vast/src"]
path = vendor/vast/src
url = https://github.com/trailofbits/vast.git
branch = mx_codegen
Copy link
Member

Choose a reason for hiding this comment

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

Is mx_codegen branch is maintained? Why not built with master branch and merge required changes from mx_codegen?

Copy link
Member

Choose a reason for hiding this comment

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

That will require multiplier to switch to llvm 19.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I made a brief attempt in 2024 to update to latest VAST, but there were a lot of issues because it started using C++ ranges heavily in a few places, and the Clang patches for PASTA/Multiplier patches fell over on them. There were other issues, even after commenting out the relevant code in VAST. Eventually I got too busy with Patchestry and gave up. I think the proper solution in the future is to re-do all or parts of Multiplier's bootstrap, but using "plain old Clang", rather than PASTA. It's also possible that there's a simple fix. Something about PASTA's Run.cpp is potentiall off.

@kumarak kumarak merged commit 1705339 into trailofbits:main Jan 17, 2025
2 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