[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