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

Adding llvm loader #107

Closed
wants to merge 1 commit into from
Closed

Adding llvm loader #107

wants to merge 1 commit into from

Conversation

siv2r
Copy link
Contributor

@siv2r siv2r commented Mar 25, 2021

Description

Initial implementation of llvm loader

Addresses #106

Type of change

Deleted checklists here will be added when this draft is ready to merge.

@siv2r siv2r force-pushed the add-llvm-loader branch 2 times, most recently from b7c3ccf to 712304b Compare March 26, 2021 15:17
added llvm_loader_test module to source/tests and renamed "mock" to "llvm"

added llvm_loader_test

edited llvm_loader

added llvm_loader as dependency to llvm_loader_test

added llvm_loader module to the source/loaders and renamed "mock" to "llvm"

added llvm_loader

added llvm_loader to CMakeLists.txt

renamed "mock" to "llvm" in llvm_loader_impl.cpp
@viferga
Copy link
Member

viferga commented May 4, 2021

I am going to close this PoC in favor of this: af60ad5

You can still continue contributing with @yashtazor ; here's his updated version of the PoC: https://github.com/yashtazor/LLVM-Loader-POC

I have adapted that version with tests to the Core.

@viferga viferga closed this May 4, 2021
@viferga viferga mentioned this pull request May 4, 2021
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