diff options
| author | Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk> | 2019-11-11 10:59:03 +0100 |
|---|---|---|
| committer | Tomaž Vajngerl <quikee@gmail.com> | 2019-11-11 22:21:38 +0100 |
| commit | c13a664c5fc1a24a5cdb4862b16d73b95ff797c7 (patch) | |
| tree | 73ead3aaa155f74d3a23ff784bc18554e085d51b /tools | |
| parent | a1cba2d4db9de4883789e9cd7669a051789a48ee (diff) | |
Don't compile cpuid test with INTRINSICS_CXXFLAGS
This will compile test with maximal supported instruction set
supported by the compiler, but the CPU might not support the
instructions sets. As this tests some aspects of runtime CPU
detection only we actually don't need to compile it with the
INTRINSICS_CXXFLAGS flags.
Change-Id: I612785949b42efbd08d1961a746025f66e99aebc
Reviewed-on: https://gerrit.libreoffice.org/82422
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/CppunitTest_tools_test.mk | 5 | ||||
| -rw-r--r-- | tools/qa/cppunit/test_cpuid.cxx | 16 |
2 files changed, 1 insertions, 20 deletions
diff --git a/tools/CppunitTest_tools_test.mk b/tools/CppunitTest_tools_test.mk index 65f6d02fd782..5672be53353f 100644 --- a/tools/CppunitTest_tools_test.mk +++ b/tools/CppunitTest_tools_test.mk @@ -30,10 +30,7 @@ $(eval $(call gb_CppunitTest_add_exception_objects,tools_test, \ tools/qa/cppunit/test_fround \ tools/qa/cppunit/test_xmlwalker \ tools/qa/cppunit/test_GenericTypeSerializer \ -)) - -$(eval $(call gb_CppunitTest_add_exception_objects,tools_test,\ - tools/qa/cppunit/test_cpuid, $(INTRINSICS_CXXFLAGS) \ + tools/qa/cppunit/test_cpuid \ )) $(eval $(call gb_CppunitTest_use_sdk_api,tools_test)) diff --git a/tools/qa/cppunit/test_cpuid.cxx b/tools/qa/cppunit/test_cpuid.cxx index fdb19d0ec133..cb9c1b9a7fb6 100644 --- a/tools/qa/cppunit/test_cpuid.cxx +++ b/tools/qa/cppunit/test_cpuid.cxx @@ -12,7 +12,6 @@ #include <cppunit/extensions/HelperMacros.h> #include <cppunit/plugin/TestPlugIn.h> #include <tools/cpuid.hxx> -#include <tools/simd.hxx> #include <rtl/ustring.hxx> namespace @@ -50,21 +49,6 @@ void CpuInstructionSetSupport::testCpuInstructionSetSupport() { CPPUNIT_ASSERT(aString.indexOf("AVX2") > 0); } - -#ifdef LO_SSE2_AVAILABLE - CPPUNIT_ASSERT_EQUAL(cpuid::hasSSE2(), - cpuid::isCpuInstructionSetSupported(cpuid::InstructionSetFlags::SSE2)); -#endif - -#ifdef LO_SSSE3_AVAILABLE - CPPUNIT_ASSERT_EQUAL(cpuid::hasSSSE3(), - cpuid::isCpuInstructionSetSupported(cpuid::InstructionSetFlags::SSE2)); -#endif - -#ifdef LO_AVX2_AVAILABLE - CPPUNIT_ASSERT_EQUAL(cpuid::hasAVX2(), - cpuid::isCpuInstructionSetSupported(cpuid::InstructionSetFlags::AVX2)); -#endif } CPPUNIT_TEST_SUITE_REGISTRATION(CpuInstructionSetSupport); |