GitHub-Proxy/eigen
1
0
Fork 0
mirror of https://gitlab.com/libeigen/eigen.git synced 2025-04-16 22:59:39 +08:00
Code Issues Packages Projects Releases Wiki Activity

bug #1684: add simplified regression test for respective clang's bug (this also reveal the same bug in Apples's clang)

Browse Source
This commit is contained in:
Gael Guennebaud 2019-02-22 10:29:06 +01:00
parent 42c23f14ac
commit 32502f3c45
1 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
test/array_reverse.cpp
View File

@ -132,6 +132,28 @@ void array_reverse_extra()
VERIFY(x.reverse() == y);
}
// Simpler version of reverseInPlace leveraging a bug
// in clang 6/7 with -O2 and AVX or AVX512 enabled.
// This simpler version ensure that the clang bug is not hidden
// through mis-inlining of reverseInPlace or other minor changes.
template<typename MatrixType>
EIGEN_DONT_INLINE
void bug1684_work(MatrixType& m1, MatrixType& m2)
{
m2 = m1;
m2.col(0).swap(m2.col(3));
m2.col(1).swap(m2.col(2));
}
template<int>
void bug1684()
{
Matrix4f m1 = Matrix4f::Random();
Matrix4f m2 = Matrix4f::Random();
bug1684_work(m1,m2);
VERIFY_IS_APPROX(m2, m1.rowwise().reverse().eval());
}
EIGEN_DECLARE_TEST(array_reverse)
{
for(int i = 0; i < g_repeat; i++) {
@ -144,6 +166,7 @@ EIGEN_DECLARE_TEST(array_reverse)
CALL_SUBTEST_7( reverse(MatrixXcd(internal::random<int>(1,EIGEN_TEST_MAX_SIZE), internal::random<int>(1,EIGEN_TEST_MAX_SIZE))) );
CALL_SUBTEST_8( reverse(Matrix<float, 100, 100>()) );
CALL_SUBTEST_9( reverse(Matrix<float,Dynamic,Dynamic,RowMajor>(internal::random<int>(1,EIGEN_TEST_MAX_SIZE), internal::random<int>(1,EIGEN_TEST_MAX_SIZE))) );
CALL_SUBTEST_3( bug1684<0>() );
}
CALL_SUBTEST_3( array_reverse_extra<0>() );
}