Apply suggestions from code review
Co-authored-by: Wesley Wiser <wwiser@gmail.com>
This commit is contained in:
parent
51268d2735
commit
b5fef37d23
@ -169,7 +169,7 @@ impl CoverageMapGenerator {
|
||||
);
|
||||
|
||||
// Create the coverage data header (Note, fields 0 and 2 are now always zero,
|
||||
// as of `llvm::coverage::CovMapVersion::Version4`.
|
||||
// as of `llvm::coverage::CovMapVersion::Version4`.)
|
||||
let zero_was_n_records_val = cx.const_u32(0);
|
||||
let filenames_size_val = cx.const_u32(filenames_size as u32);
|
||||
let zero_was_coverage_size_val = cx.const_u32(0);
|
||||
|
@ -56,7 +56,7 @@ else
|
||||
-DINSTR_PROF_ORDERFILE='$(DATA_SECTION_PREFIX)__llvm_orderfile'
|
||||
endif
|
||||
|
||||
ifeq ($(LLVM_VERSION_11_PLUS),false)
|
||||
ifeq ($(LLVM_VERSION_11_PLUS),true)
|
||||
all: test_llvm_ir
|
||||
else
|
||||
$(info Rust option `-Z instrument-coverage` requires LLVM 11 or higher. Test skipped.)
|
||||
|
Loading…
x
Reference in New Issue
Block a user