summaryrefslogtreecommitdiff
path: root/sal
diff options
context:
space:
mode:
authorLuboš Luňák <l.lunak@suse.cz>2012-11-15 18:07:06 +0100
committerLuboš Luňák <l.lunak@suse.cz>2012-11-15 18:13:56 +0100
commit6adc24cd861cdde45d8f55943a836246470fa72c (patch)
tree2fbc4b50a93c347dbfb64ff49269891f877e2646 /sal
parentad5a7fbf8eb3be526a2c235fb41aba5792d806ec (diff)
Revert "use intrinsict to inline InterlockCount() on Windows too"
This reverts commit 5ae27819974e3492b6f4d33fa51ef2a9d77b5ce5. The commit breaks smoketest on Windows, for reasons unknown: C:/cygwin/home/tinderbox/tinderbox-master-build-libreoffice/smoketest/smoketest.cxx(197) : error : Assertion Test name: `anonymous namespace'::Test::test assertion failed - Expression: connection_.isStillAlive() I can't see why the commit should matter, given that the functions should be equivalent according to my reading of the docs. And debugging this on Windows is pain squared, so I can't even figure out what is going wrong where. Feel free to try your luck. Note that when debugging this, the code triggering this seems to be run only if SOLAR_JAVA and JAVA_HOME are set, so 'make verbose=t' and running just the command without this hides the problem.
Diffstat (limited to 'sal')
-rw-r--r--sal/inc/osl/interlck.h4
-rw-r--r--sal/inc/sal/config.h2
2 files changed, 0 insertions, 6 deletions
diff --git a/sal/inc/osl/interlck.h b/sal/inc/osl/interlck.h
index 22ff98504eae..792ea55051d9 100644
--- a/sal/inc/osl/interlck.h
+++ b/sal/inc/osl/interlck.h
@@ -67,8 +67,6 @@ SAL_DLLPUBLIC oslInterlockedCount SAL_CALL osl_decrementInterlockedCount(oslInte
*/
#if defined( HAVE_GCC_BUILTIN_ATOMIC )
# define osl_atomic_increment(p) __sync_add_and_fetch((p), 1)
-#elif defined( _MSC_VER )
-# define osl_atomic_increment(p) _InterlockedIncrement((p))
#else
# define osl_atomic_increment(p) osl_incrementInterlockedCount((p))
#endif
@@ -86,8 +84,6 @@ SAL_DLLPUBLIC oslInterlockedCount SAL_CALL osl_decrementInterlockedCount(oslInte
*/
#if defined( HAVE_GCC_BUILTIN_ATOMIC )
# define osl_atomic_decrement(p) __sync_sub_and_fetch((p), 1)
-#elif defined( _MSC_VER )
-# define osl_atomic_decrement(p) _InterlockedDecrement((p))
#else
# define osl_atomic_decrement(p) osl_decrementInterlockedCount((p))
#endif
diff --git a/sal/inc/sal/config.h b/sal/inc/sal/config.h
index e81250ac0351..aee8e829067b 100644
--- a/sal/inc/sal/config.h
+++ b/sal/inc/sal/config.h
@@ -44,8 +44,6 @@
#ifdef _MSC_VER
-#include <Intrin.h> // define intrinsic functions supported by the compiler
-
#ifndef _USE_MATH_DEFINES
#define _USE_MATH_DEFINES // needed by Visual C++ for math constants
#endif