Skip to content

Move coverage tests from run-make-fulldeps to run-make #83830

Closed

Description

@richkadel

I tried to do this recently and it failed. Some things to remember:

  • Search for and replace references to run-make-fulldeps
  • Update path to ../../run-make-fulldeps/tools.mk from just ../tools.mk
  • bootstrap has an explicit reference to fulldeps in test.rs to build RustDemangler and set --rust-demangler-path, which should be changed to support the move to run-make without fulldeps

I forgot about that last one and its possible that was the only issue.

If not, check bootstrap for other fulldeps dependencies, and note that the error I thought I saw had to do with llvm-profdata or llvm-cov. If so, those may require additional special handling in bootstrap or elsewhere.

Activity

  1. added 2 commits that reference this issue on May 1, 2021
    3fca198
  2. added 7 commits that reference this issue on May 4, 2021
    438bf55
    43ecb23
    899ea84
    5d5ba5a
    f189603
    5588f10
    7d26d3b
  3. richkadel commented on May 19, 2021

    @richkadel
    Author

    Note that additional attempts to migrate coverage tests from run-make-fulldeps to run-make did not succeed because run-make runs on some platforms that don't support LLVM coverage profiling.

    See PR #85007

    I'm closing this issue as "Not Feasible" at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions