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

Remove bogus assert
Some checks failed
Tests / Clang total: 3296, passed: 3296
Details
Clang |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |0|0|0|0|:clap:
Details
Tests / 64 bit versions total: 3296, passed: 3296
Details
Tests / Debug total: 3294, passed: 3294
Details
Tests / SIMD fallback total: 3296, passed: 3296
Details
Tests / Release [gcc] total: 3296, passed: 3296
Details
GNU C Compiler (gcc) |Total|New|Outstanding|Fixed|Trend |:-:|:-:|:-:|:-:|:-: |2|0|2|0|:zzz:
Details
Tests / Release [gcc,aarch64] total: 2458, passed: 2458
Details
Tests / Coverage total: 2476, passed: 2476
Details
Code Coverage #### Project Overview No changes detected, that affect the code coverage. * Line Coverage: 98.77% (1841/1864) * Branch Coverage: 67.55% (1445/2139) * Complexity Density: 0.00 * Lines of Code: 1864 #### Quality Gates Summary Output truncated.
Details
weaselab/conflict-set/pipeline/head There was a failure building this commit
Details

Browse Source
This commit is contained in:
Andrew Noyes
2024-09-14 10:58:48 -07:00
parent 52201fa4c7
commit d9e4a7d1b6
1 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
ConflictSet.cpp
View File

@@ -2925,17 +2925,11 @@ horizontalMaxUpTo16(InternalVersionT *vs, InternalVersionT z, int len) {
uint32_t zero;
memcpy(&zero, &z, sizeof(zero));
auto zeroVec = _mm512_set1_epi32(zero);
auto actual = InternalVersionT(
auto max = InternalVersionT(
zero +
_mm512_reduce_max_epi32(_mm512_sub_epi32(
_mm512_mask_loadu_epi32(zeroVec, _mm512_int2mask((1 << len) - 1), vs),
zeroVec)));
// Hope it gets vectorized
InternalVersionT max = vs[0];
for (int i = 1; i < len; ++i) {
max = std::max(vs[i], max);
}
assert(actual == max);
return max;
#endif
}