weaselab/conflict-set
2
0
Fork 0
Code Issues 4 Packages Releases 13 Activity

Only exclude avx512-targeted functions from coverage
Some checks failed
Tests / Clang total: 1479, passed: 1479
Details
Clang |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |0|0|0|0|:clap:
Details
Tests / Debug total: 1477, passed: 1477
Details
Tests / SIMD fallback total: 1479, passed: 1479
Details
weaselab/conflict-set/pipeline/head There was a failure building this commit
Details

Browse Source
This commit is contained in:
Andrew Noyes
2024-08-02 20:53:30 -07:00
parent abce4591d0
commit 493a6572ad
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
ConflictSet.cpp
View File

@@ -1673,6 +1673,8 @@ int firstNeqStride(const uint8_t *ap, const uint8_t *bp) {
#endif #endif
} }
// This gets covered in local development
// GCOVR_EXCL_START
#if defined(HAS_AVX) && !defined(__SANITIZE_THREAD__) #if defined(HAS_AVX) && !defined(__SANITIZE_THREAD__)
__attribute__((target("avx512bw"))) int __attribute__((target("avx512bw"))) int
longestCommonPrefix(const uint8_t *ap, const uint8_t *bp, int cl) { longestCommonPrefix(const uint8_t *ap, const uint8_t *bp, int cl) {
@@ -1699,6 +1701,7 @@ longestCommonPrefix(const uint8_t *ap, const uint8_t *bp, int cl) {
} }
__attribute__((target("default"))) __attribute__((target("default")))
#endif #endif
// GCOVR_EXCL_STOP
int longestCommonPrefix(const uint8_t *ap, const uint8_t *bp, int cl) { int longestCommonPrefix(const uint8_t *ap, const uint8_t *bp, int cl) {
assume(cl >= 0); assume(cl >= 0);
@@ -1941,9 +1944,6 @@ downLeftSpine:
return n->entry.rangeVersion <= readVersion; return n->entry.rangeVersion <= readVersion;
} }
// Only one of these is ever exercised. I'm not worried about somehow not
// calling one of these though.
// GCOVR_EXCL_START
#ifdef HAS_AVX #ifdef HAS_AVX
uint32_t compare16_32bit(const InternalVersionT *vs, InternalVersionT rv) { uint32_t compare16_32bit(const InternalVersionT *vs, InternalVersionT rv) {
uint32_t compared = 0; uint32_t compared = 0;
@@ -1961,6 +1961,8 @@ uint32_t compare16_32bit(const InternalVersionT *vs, InternalVersionT rv) {
return compared; return compared;
} }
// This gets covered in local development
// GCOVR_EXCL_START
__attribute__((target("avx512f"))) uint32_t __attribute__((target("avx512f"))) uint32_t
compare16_32bit_avx512(const InternalVersionT *vs, InternalVersionT rv) { compare16_32bit_avx512(const InternalVersionT *vs, InternalVersionT rv) {
uint32_t r; uint32_t r;