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

Using llvm-config creates useless rebuilds #1412

Closed
llvmbot opened this issue Dec 10, 2006 · 4 comments
Closed

Using llvm-config creates useless rebuilds #1412

llvmbot opened this issue Dec 10, 2006 · 4 comments
Labels
bugzilla Issues migrated from bugzilla build-problem

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Dec 10, 2006

Bugzilla Link 1040
Resolution FIXED
Resolved on Feb 22, 2010 12:51
Version 1.6
OS All
Reporter LLVM Bugzilla Contributor
CC @lattner

Extended Description

Build times have increased after the tools were changed to use llvm-config to
generate the list of libraries to link with. Here's the scenario:

  1. llvm-config depends on all libraries so a change to any one of them results
    in llvm-config being rebuilt.

  2. All tools depend on llvm-config so any change to it results in all tools being
    rebuilt.

For example, consider a change to lib/Transforms/Scalar. It should only affect
opt, gccas, gccld, and maybe llc and lli. However, when you rebuild, all tools
are relinked. There's no reason that llvm-dis, llvm-as, etc. need to be
relinked. Previously the makefiles did not re-link these tools for which a
dependency was not changed.

Its unclear what the correct solution is. We need an "existence" dependency for
the tools on llvm-config.

@lattner
Copy link
Collaborator

lattner commented Dec 10, 2006

the fix is for 'building' llvm-config to not actually change it if the contents don't change. This is exactly
what we do for tblgen. If you change tblgen's source in a way that doesn't affect it's output (e.g. change
libsupport and relink tblgen), all the targets aren't rebuilt.

-Chris

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 10, 2006

Good idea. I'll try to implement that when I get a chance.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 10, 2006

Mine.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 12, 2006

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla build-problem
Projects
None yet
Development

No branches or pull requests

2 participants