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

Tue Feb 23 08:23:42 PST 2021

jsji added a comment.

In D95753#2581150 <https://reviews.llvm.org/D95753#2581150>, @phosek wrote:

> 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.

Also seeing similar failure on PowerPC RHEL buildbot. http://lab.llvm.org:8011/#/builders/57/builds/4516

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95753