-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[make install] Shouldn't install utils, examples or projects/sample #1100
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
Comments
assigned to @lattner |
So, how about "make full-install" to get everything and default "make install" |
For the record, "make full-install" and "make install" sound great :) |
Okay, one question on this: When you say you don't want the "utils" installed with "make install", then I'm |
llvm-test isn't run with an installed tree, so that's not an issue (it uses fpcmp). tblgen is used by targets outside the tree, if they could possible exist. I don't think it works and I'm At least at apple, people don't use 'make install'ed trees to do development of LLVM libraries (e.g. code Do you really see any reason people would be compelled to build a target out of tree? Building a target -Chris |
I'm not attached to the outcome. I was just asking a question. I had no idea how |
ok, sounds good. Don't forget includes :) Thanks Reid! -Chris |
This is turning into quite a bit of work and so I'm going to combine it with
Consequently, this bug won't get resolved until (probably) after 1.7 is shipped. |
Sounds great. I like that design. Thanks, -Chris |
This isn't slated for 1.7 |
This patch implements transformations for VAArg in X86_64 ABI **in shape**. `In shape` means it can't work properly due to the dependent X86_64 ABI is not robust. e.g., when we want to use VAArg with `long double`, we need llvm/clangir#1087. This patch literally implement https://github.com/llvm/llvm-project/blob/d233fedfb0de882353c348cd1ac57dab619efa6d/clang/lib/CodeGen/Targets/X86.cpp#L3015-L3240 in CIR. There some differences due to the traditional pipeline are converting AST to LLVM and we're transforming CIR to CIR. And also to get the ABI Info, I moved `X86_64ABIInfo` to the header.
Extended Description
It would be nice if make install (or make install-something-else) only installed the LLVM libraries and
tools, not llvm/utils, llvm/examples and llvm/projects.
The utils is only used when building LLVM (not by clients who use the llvm headers/libraries), the others
are only used by developers not by people looking to link to llvm libraries and use tools.
-Chris
The text was updated successfully, but these errors were encountered: