dblaikie added inline comments.
================ Comment at: llvm/test/tools/llvm-cov/branch-noShowBranch.test:3 +// RUN: llvm-profdata merge %S/Inputs/branch-c-general.proftext -o %t.profdata +// RUN: llvm-cov show %S/Inputs/branch-c-general.o32l -instr-profile %t.profdata -path-equivalence=/tmp,%S %S/branch-c-general.c | FileCheck %s +// RUN: llvm-cov report %S/Inputs/branch-c-general.o32l --show-branch-summary=false -instr-profile %t.profdata -show-functions -path-equivalence=/tmp,%S %S/branch-c-general.c | FileCheck %s -check-prefix=REPORT ---------------- This test depends on another test file as input which is generally not done in the LLVM test suite - inputs to tests are placed in the "Inputs" subdirectory, rather than in the test directory itself. I realize a few other tests already here (demangle, hideUnexpectedSubviews, style, and threads) already do this - but it'd be good to not add more cases (& fix those existing cases where possible) Could you fix this so this test (& possibly others, if you have the opportunity/bandwidth) doesn't depend on other test files, but only on files in the Inputs subdirectory? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84467/new/ https://reviews.llvm.org/D84467 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
