[cmake-developers] Proposal to disable link depends inference.
Brad King
brad.king at kitware.com
Fri Apr 21 10:08:59 EDT 2017
On 04/20/2017 05:16 PM, Matthew Hanna (BLOOMBERG/ 731 LEX) wrote:
> avoid adding the external to external edges
> https://gitlab.kitware.com/mhanna21/cmake/commit/2c7d2e33b6218e58f88957fb793ce0a781ec76aa
That sounds good. From the patch:
> + if (this->EntryList[depender_index].Target ||
> + this->EntryList[(int)*edge].Target) {
> + edges.push_back(*edge);
> + }
I think only the second condition is needed. To check, try adding:
assert(!this->EntryList[depender_index].Target);
there. I don't think internal targets can get an InferredDependSets
entry, according to logic in `cmComputeLinkDepends::AddLinkEntry`.
-Brad
More information about the cmake-developers
mailing list