Merge pull request #3214 from eseiler/fix/perf

[FIX] Disable SeqAn2 OpenMP benchmark on clang
seqan · Nov 16, 2023 · 4c10afc · 4c10afc · vercel · Nov 16, 2023
2 parents 5cbcaf4 + d546f4b
commit 4c10afc
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/performance/alignment/global_affine_alignment_parallel_benchmark.cpp b/test/performance/alignment/global_affine_alignment_parallel_benchmark.cpp
@@ -160,7 +160,8 @@ void seqan2_affine_dna4_parallel(benchmark::State & state)
 BENCHMARK_TEMPLATE(seqan2_affine_dna4_parallel, score)->UseRealTime();
 #endif // SEQAN3_HAS_SEQAN2
 
-#if defined(SEQAN3_HAS_SEQAN2) && defined(_OPENMP)
+// Crashes with libc++
+#if defined(SEQAN3_HAS_SEQAN2) && defined(_OPENMP) && !defined(_LIBCPP_VERSION)
 template <typename result_t>
 void seqan2_affine_dna4_omp_for(benchmark::State & state)
 {
@@ -201,7 +202,7 @@ void seqan2_affine_dna4_omp_for(benchmark::State & state)
 
 BENCHMARK_TEMPLATE(seqan2_affine_dna4_omp_for, score)->UseRealTime();
 BENCHMARK_TEMPLATE(seqan2_affine_dna4_omp_for, trace)->UseRealTime();
-#endif // defined(SEQAN3_HAS_SEQAN2) && defined(_OPENMP)
+#endif // defined(SEQAN3_HAS_SEQAN2) && defined(_OPENMP) && !defined(_LIBCPP_VERSION)
 
 // ============================================================================
 //  instantiate tests