[llvm-commits] [llvm] r119355 - /llvm/trunk/cmake/modules/LLVMLibDeps.cmake

[Óscar Fuentes]

Wesley Peck <peckw at wesleypeck.com> writes:

> Is LLVMLibDeps.cmake something that should not be checked in when
> changed?

In theory, yes.

> I've noticed that it almost always gets changed when I build
> LLVM but I have been unsure if I should commit this file when it
> changes.

There is some problem with GenLibDeps.pl reporting different
dependencies on consecutive builds. It usually affects the asmprinters
and asmparsers. Furthermore, sometimes the build fails with error
reports about missing vtables. I don't have a clear idea about its
origin. It is *really* annoying.

For now on, unless the diff of LLVMLibDeps.cmake shows a dependency
change related to the code you edited, don't bother committing
LLVMLibDeps.cmake.