[PATCH] D124490: [InstrProf] Minimal Block Coverage

Ellis Hoag via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 16:56:15 PDT 2022


ellis updated this revision to Diff 434207.
ellis added a comment.

Remove logic that enforced a non-instrumented node to either depend on its predecessors or successors, but not both. This logic was valid, but not necessary to produce a valid coverage instrumentation, so it was removed for simplicity.

Also, copy the interesting test case from compiler-rt to llvm. This allows us to more easily debug the complex test case in LLVM rather than compiler-rt.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124490

Files:
  compiler-rt/test/profile/instrprof-coverage.c
  compiler-rt/test/profile/instrprof-entry-coverage.c
  llvm/include/llvm/Transforms/Instrumentation/BlockCoverageInference.h
  llvm/lib/Transforms/Instrumentation/BlockCoverageInference.cpp
  llvm/lib/Transforms/Instrumentation/CMakeLists.txt
  llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
  llvm/test/Transforms/PGOProfile/coverage.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124490.434207.patch
Type: text/x-patch
Size: 43039 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220603/1d581549/attachment-0001.bin>


More information about the llvm-commits mailing list