Skip to content

Conversation

@asthetik
Copy link

This PR fixes issue #1116 by passing the --threads parameter from grcov to the underlying llvm-profdata merge command

@marco-c
Copy link
Collaborator

marco-c commented Sep 8, 2025

A couple of things:

  1. we should do the same for llvm-cov export;
  2. since llvm-cov and llvm-profdata actually do this by default when --num-threads is 0 (and they autodetect the right number of threads to use), if the grcov parameter is not passed explicitly by the user, let's rely on their autodetection instead of ours.

@marco-c marco-c linked an issue Sep 8, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

grcov --threads parameter is not passed to sub commands

2 participants