Skip to content

Commit cbb810d

Browse files
authored
Merge pull request #2294 from BioDataAnalysis/bda_clang_xrandom_shuffle
test/test_xrandom.cpp: Disabled a failling shuffle test also for Clang
2 parents af28d7f + e4d43ad commit cbb810d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/test_xrandom.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,9 +203,9 @@ namespace xt
203203
auto ar = a;
204204

205205
xt::random::seed(123);
206-
// Unfortunately MSVC & OS X seem to produce different shuffles even though the
207-
// generated integer sequence should be the same ...
208-
#ifdef __linux__
206+
// Unfortunately MSVC, OS X, and Clang on Linux seem to produce different
207+
// shuffles even though the generated integer sequence should be the same ...
208+
#if defined(__linux__) && (!defined(__clang__) || (__clang_major__ < 11))
209209
xt::random::shuffle(a);
210210
EXPECT_EQ(xt::view(ar, keep(0, 1, 2)), a);
211211
xt::random::shuffle(a);

0 commit comments

Comments
 (0)