[PATCH] D95753: [Coverage] Store compilation dir separately in coverage mapping

[Petr Hosek via Phabricator via cfe-commits]

phosek added a comment.

In D95753#2578724 <https://reviews.llvm.org/D95753#2578724>, @gregmiller wrote:

> Hello, we are seeing failures like below:
>
> Note: Google Test filter = ParameterizedCovMapTest/CoverageMappingTest.skip_duplicate_function_record/0
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ParameterizedCovMapTest/CoverageMappingTest
> [ RUN ] ParameterizedCovMapTest/CoverageMappingTest.skip_duplicate_function_record/0
> /scratch/build_jenkins/workspace/Downstream_Changes/llvm_cgt/llvm-project/llvm/unittests/ProfileData/CoverageMappingTest.cpp:889: Failure
> Expected: 3U
> Which is: 3
> To be equal to: NumFuncs
> Which is: 5
> [ FAILED ] ParameterizedCovMapTest/CoverageMappingTest.skip_duplicate_function_record/0, where GetParam() = (false, false) (0 ms)
>
> Do you have any suggestions on why we would start seeing above failures after your commit? Was passing before.
>
> Thanks
> Greg

That's really unexpected. What's your build configuration? Is there any way for me to reproduce this failure locally? I'd like to take a closer look.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95753/new/

https://reviews.llvm.org/D95753