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

Removing the need for llvm_ext.cc #13946

Closed
6 tasks done
HertzDevil opened this issue Nov 3, 2023 · 2 comments
Closed
6 tasks done

Removing the need for llvm_ext.cc #13946

HertzDevil opened this issue Nov 3, 2023 · 2 comments

Comments

@HertzDevil
Copy link
Contributor

HertzDevil commented Nov 3, 2023

If everything we need is already in LLVM's C API, then our build process will be considerably simpler (see for example #10880), and it'll become possible to build + distribute a Windows compiler that dynamically links against LLVM-C.dll without any LLVM static libraries.

The following list is mostly copied from #13177:

@HertzDevil
Copy link
Contributor Author

As a final step, we need a way to tell the Makefile(s) that llvm_ext.obj doesn't need to be built at all and that the local compiler doesn't depend on it. This would happen if LLVM_VERSION is new enough, or if we explicitly disable it via a flag.

@straight-shoota
Copy link
Member

With #14357 we can consider this resolved. llvm_ext is no longer necessary when linking LLVM 18+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants