Skip to content
This repository was archived by the owner on May 21, 2025. It is now read-only.
This repository was archived by the owner on May 21, 2025. It is now read-only.

Add support for clang and llvm-cov #52

Open

Description

@nijel

It would be great if coverage would work for llvm as well out of the box. It provides quite similar interface to gcov, it just has to be executed as llvm-cov gcov, see http://llvm.org/docs/CommandGuide/llvm-cov.html

It's usage could be specified by parameter or autodected based on CC environment (use llvm-cov if CC is clang).

Activity

  1. changed the title [-]Add support for llvm-cov[/-] [+]Add support for clang and llvm-cov[/+] on Jan 4, 2016
  2. stevepeak commented on Jan 7, 2016

    @stevepeak
    Contributor

    @nijel would love to. Do you have any open source projects showing off how llvm is used so I can review its process. Thanks!

  3. nijel commented on Jan 8, 2016

    @nijel
    Author

    I use it to build some projects on Travis CI, but meanwhile I've realized that the LLVM version there is quite old and the llvm-cov is probably not flexible enough to do what is needed.

    But with recent version, all what should be needed is to run llvm-cov gcov instead of gcov. It seems to support all needed options...

  4. stevepeak commented on Feb 5, 2016

    @stevepeak
    Contributor

    @nijel perhaps other customers are having related issues, even though I have not heard of any to my knowledge. Do you have any suggestions on how to "automatically" detect when we should use llvm-cov gcov instead of gcov?

    https://github.com/codecov/codecov-bash/blob/346e91cb2918fd1d84fde46c0814ae2005fee363/codecov#L484

    You can already specify a custom "gcov" via -x "llvm-cov gcov"

  5. nijel commented on Feb 8, 2016

    @nijel
    Author

    The only idea is to base this on CC environment variable. That should work quite reliable for Travis CI, but not sure about other environments...

  6. stevepeak commented on Feb 23, 2016

    @stevepeak
    Contributor

    @nijel I'm not familiar with the working of llvm-cov, but would invite a pull request to review your suggestions on how to implement this. Thanks!

  7. nijel commented on Mar 4, 2016

    @nijel
    Author

    I will look into this once Travis has upgraded LLVM to more recent version which behaves better in regards to coverage.

  8. self-assigned this
    on Nov 29, 2020
  9. removed their assignment
    on Jul 21, 2021
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