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

Remove llvm::InitializeAll* calls #468

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

wenju-he
Copy link
Contributor

@wenju-he wenju-he commented Aug 1, 2023

opencl-clang only uses clang libraries to generate frontend IR and
explicitly disables llvm optimizations. In addition, opencl-clang
doesn't parse/print assembly.
Therefore, there is no need to initialize llvm target machines, target
MCs, asm printers and asm parsers.

This PR removes opencl-clang's dependency on some llvm libraries.

opencl-clang only uses clang libraries to generate frontend IR and
explicitly disables llvm optimizations. In addition, opencl-clang
doesn't parse/print assembly.
Therefore, there is no need to initialize llvm target machines, target
MCs, asm printers and asm parsers

This PR removes opencl-clang's dependency on some llvm libraries.
@wenju-he wenju-he requested review from cdai2 and haonanya August 1, 2023 04:18
@wenju-he wenju-he merged commit 44edebc into intel:main Aug 1, 2023
2 checks passed
@wenju-he wenju-he deleted the InitializeAllTargets branch August 1, 2023 06:40
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