[PATCH] D138846: MC/DC in LLVM Source-Based Code Coverage: LLVM back-end and compiler-rt
Alan Phipps via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 29 14:24:37 PST 2022
alanphipps added inline comments.
================
Comment at: llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp:795
+void InstrProfiling::lowerMCDCTestVectorBitmapUpdate(
+ InstrProfMCDCTVBitmapUpdate *Update) {
----------------
ellis wrote:
> I'm not seeing any tests in `llvm/test/Instrumentation/InstrProfiling/` to test that this code was lowered correctly. Can you add some?
>
> It would also be good to add tests in `clang/test/Profile` or `llvm/test/Transforms/PGOProfile/` to make sure these intrinsics are added to the correct locations (I actually don't see where `InstrProfMCDCTVBitmapUpdate` is created).
Hmm. Right, I do test the lowering using tests I added in `clang/test/Profile` (It's in the clang-specific review, but obviously it has overlap with what is done in this portion), but I overlooked adding anything specific to `llvm/test/Instrumentation/InstrProfiling`.
I'll look into adding some additional tests for this, as well as for the intrinsics.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138846/new/
https://reviews.llvm.org/D138846
More information about the llvm-commits
mailing list