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

Wrap BC generation in flag #1

Open
wants to merge 4 commits into
base: llvm-19
Choose a base branch
from

Conversation

kyle-elliott
Copy link

Helps windows users compile and link on windows, however they will still need to pull the generated BC files from an external source to use the library properly

Helps windows users compile and link on windows, however they will still need to pull the generated BC files from an external source to use the library properly
@kyle-elliott kyle-elliott marked this pull request as draft November 13, 2024 18:31
@kyle-elliott kyle-elliott marked this pull request as ready for review November 13, 2024 20:43
@kyle-elliott
Copy link
Author

Commenting some addition info for any users in the future attempting to compile on windows, even using clang-cl with long double type set to a size of 80 bits, you will fail to compile due to there not being a specialization of std::bit_cast with 80 bit support.

Following fork contains general solutions to any issues you face
https://github.com/Colton1skees/remill/commits/system-llvm-include/

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.

1 participant