llvm-cov: merging reports with multiple definitions of the same symbol |
|||
Issue descriptionSee http://lists.llvm.org/pipermail/llvm-dev/2018-March/121705.html Doesn't seem to be a blocker at this point, but might worth to fix in future, or at least investigate what we can do about it.
,
May 10 2018
I've landed the fix upstream (https://reviews.llvm.org/rL331801), waiting for the next clang roll now.
,
May 17 2018
Verified that we now see all _fuzzer files now. e.g. https://chromium-coverage.appspot.com/reports/559247/linux/chromium/src/base/json/json_reader_fuzzer.cc.html https://chromium-coverage.appspot.com/reports/559247/linux/chromium/src/base/json/json_correctness_fuzzer.cc.html https://chromium-coverage.appspot.com/reports/559247/linux/chromium/src/base/json/string_escape_fuzzer.cc.html
,
May 17 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6b33c6d64fbb9582f5f25086e3475c31ef30f484 commit 6b33c6d64fbb9582f5f25086e3475c31ef30f484 Author: Max Moroz <mmoroz@chromium.org> Date: Thu May 17 14:04:21 2018 [Code Coverage] Remove obsolete points from the FAQ list: - Entry regarding coverage missing for some *_fuzzer.cc files. - Note regarding coverage from child processes from the browser. TBR=inferno@chromium.org Bug: 821617 , 834781 Change-Id: I69a645e3b311ac9c160d878afd518706b3a000fe Reviewed-on: https://chromium-review.googlesource.com/1063995 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by: Max Moroz <mmoroz@chromium.org> Reviewed-by: Abhishek Arya <inferno@chromium.org> Cr-Commit-Position: refs/heads/master@{#559516} [modify] https://crrev.com/6b33c6d64fbb9582f5f25086e3475c31ef30f484/docs/code_coverage.md |
|||
►
Sign in to add a comment |
|||
Comment 1 by mmoroz@chromium.org
, May 10 2018