diff --git a/TracyClient.cpp b/TracyClient.cpp index 273ccae0..8f465650 100644 --- a/TracyClient.cpp +++ b/TracyClient.cpp @@ -11,10 +11,6 @@ // Define TRACY_ENABLE to enable profiler. -#if ( defined _MSC_VER || defined __CYGWIN__ ) && !defined WIN32_LEAN_AND_MEAN -# define WIN32_LEAN_AND_MEAN -#endif - #include "common/TracySystem.cpp" #ifdef TRACY_ENABLE diff --git a/client/TracyCallstack.cpp b/client/TracyCallstack.cpp index 8e549b65..e76d24a2 100644 --- a/client/TracyCallstack.cpp +++ b/client/TracyCallstack.cpp @@ -5,7 +5,10 @@ #if TRACY_HAS_CALLSTACK == 1 # include +# pragma warning( push ) +# pragma warning( disable : 4091 ) # include +# pragma warning( pop ) #elif TRACY_HAS_CALLSTACK >= 2 # include # include diff --git a/client/TracyProfiler.cpp b/client/TracyProfiler.cpp index 33e9c41a..bf26292f 100644 --- a/client/TracyProfiler.cpp +++ b/client/TracyProfiler.cpp @@ -85,7 +85,7 @@ static int SetupHwTimerFailed() return sigsetjmp( SigIllEnv, 1 ); } -static void SetupHwTimerSigIllHandler( int signum ) +static void SetupHwTimerSigIllHandler( int /*signum*/ ) { siglongjmp( SigIllEnv, 1 ); } @@ -124,6 +124,7 @@ static int64_t SetupHwTimer() static const char* GetProcessName() { + const char* processName = "unknown"; #if defined _MSC_VER static char buf[_MAX_PATH]; GetModuleFileNameA( nullptr, buf, _MAX_PATH ); @@ -131,16 +132,16 @@ static const char* GetProcessName() while( *ptr != '\0' ) ptr++; while( ptr > buf && *ptr != '\\' && *ptr != '/' ) ptr--; if( ptr > buf ) ptr++; - return ptr; + processName = ptr; #elif defined __ANDROID__ # if __ANDROID_API__ >= 21 auto buf = getprogname(); - if( buf ) return buf; + if( buf ) processName = buf; # endif #elif defined _GNU_SOURCE || defined __CYGWIN__ - return program_invocation_short_name; + processName = program_invocation_short_name; #endif - return "unknown"; + return processName; } enum { QueuePrealloc = 256 * 1024 }; @@ -728,7 +729,7 @@ void Profiler::SendCallstackPayload( uint64_t _ptr ) AppendDataUnsafe( &item, QueueDataSize[(int)QueueType::CallstackPayload] ); AppendDataUnsafe( &l16, sizeof( l16 ) ); - if( sizeof( uintptr_t ) == sizeof( uint64_t ) ) + if( compile_time_condition::value ) { AppendDataUnsafe( ptr, sizeof( uint64_t ) * sz ); } diff --git a/client/TracyProfiler.hpp b/client/TracyProfiler.hpp index bb11fb4e..58e882c1 100644 --- a/client/TracyProfiler.hpp +++ b/client/TracyProfiler.hpp @@ -373,7 +373,7 @@ private: MemWrite( &item->memAlloc.time, GetTime() ); MemWrite( &item->memAlloc.thread, thread ); MemWrite( &item->memAlloc.ptr, (uint64_t)ptr ); - if( sizeof( size ) == 4 ) + if( compile_time_condition::value ) { memcpy( &item->memAlloc.size, &size, 4 ); memset( &item->memAlloc.size + 4, 0, 2 ); diff --git a/client/concurrentqueue.h b/client/concurrentqueue.h index 7a57e547..a3d0237f 100644 --- a/client/concurrentqueue.h +++ b/client/concurrentqueue.h @@ -231,6 +231,21 @@ namespace moodycamel { namespace details { #endif } } +namespace +{ + // to avoid MSVC warning 4127: conditional expression is constant + template + struct compile_time_condition + { + static const bool value = false; + }; + template <> + struct compile_time_condition + { + static const bool value = true; + }; +} + #ifdef MOODYCAMEL_QUEUE_INTERNAL_DEBUG #include "internal/concurrentqueue_internal_debug.h" #endif @@ -797,7 +812,7 @@ public: } // Destroy implicit producer hash tables - if (INITIAL_IMPLICIT_PRODUCER_HASH_SIZE != 0) { + if (compile_time_condition::value) { auto hash = implicitProducerHash.load(std::memory_order_relaxed); while (hash != nullptr) { auto prev = hash->prev; @@ -1504,7 +1519,7 @@ private: template inline bool is_empty() const { - if (context == explicit_context && BLOCK_SIZE <= EXPLICIT_BLOCK_EMPTY_COUNTER_THRESHOLD) { + if (compile_time_condition::value) { // Check flags for (size_t i = 0; i < BLOCK_SIZE; ++i) { if (!emptyFlags[i].load(std::memory_order_relaxed)) { @@ -1550,7 +1565,7 @@ private: template inline bool set_many_empty(index_t i, size_t count) { - if (context == explicit_context && BLOCK_SIZE <= EXPLICIT_BLOCK_EMPTY_COUNTER_THRESHOLD) { + if (compile_time_condition::value) { // Set flags std::atomic_thread_fence(std::memory_order_release); i = BLOCK_SIZE - 1 - static_cast(i & static_cast(BLOCK_SIZE - 1)) - count + 1; @@ -1586,7 +1601,7 @@ private: template inline void reset_empty() { - if (context == explicit_context && BLOCK_SIZE <= EXPLICIT_BLOCK_EMPTY_COUNTER_THRESHOLD) { + if (compile_time_condition::value) { // Reset flags for (size_t i = 0; i != BLOCK_SIZE; ++i) { emptyFlags[i].store(false, std::memory_order_relaxed); @@ -3298,7 +3313,7 @@ private: inline void populate_initial_implicit_producer_hash() { - if (INITIAL_IMPLICIT_PRODUCER_HASH_SIZE == 0) return; + if (compile_time_condition::value) return; implicitProducerHashCount.store(0, std::memory_order_relaxed); auto hash = &initialImplicitProducerHash; @@ -3640,7 +3655,7 @@ ConsumerToken::ConsumerToken(ConcurrentQueue& queue) : itemsConsumedFromCurrent(0), currentProducer(nullptr), desiredProducer(nullptr) { initialOffset = queue.nextExplicitConsumerId.fetch_add(1, std::memory_order_release); - lastKnownGlobalOffset = -1; + lastKnownGlobalOffset = static_cast(-1); } template @@ -3648,7 +3663,7 @@ ConsumerToken::ConsumerToken(BlockingConcurrentQueue& queue) : itemsConsumedFromCurrent(0), currentProducer(nullptr), desiredProducer(nullptr) { initialOffset = reinterpret_cast*>(&queue)->nextExplicitConsumerId.fetch_add(1, std::memory_order_release); - lastKnownGlobalOffset = -1; + lastKnownGlobalOffset = static_cast(-1); } template diff --git a/client/tracy_rpmalloc.cpp b/client/tracy_rpmalloc.cpp index 08e075f9..10957fd3 100644 --- a/client/tracy_rpmalloc.cpp +++ b/client/tracy_rpmalloc.cpp @@ -85,6 +85,8 @@ /// Platform and arch specifics #ifdef _MSC_VER +# pragma warning( push ) +# pragma warning( disable : 4324 ) # define ALIGNED_STRUCT(name, alignment) __declspec(align(alignment)) struct name # define FORCEINLINE __forceinline # define atomic_thread_fence_acquire() //_ReadWriteBarrier() @@ -1769,13 +1771,13 @@ _memory_map_os(size_t size, size_t* offset) { //Ok to MEM_COMMIT - according to MSDN, "actual physical pages are not allocated unless/until the virtual addresses are actually accessed" void* ptr = VirtualAlloc(0, size + padding, MEM_RESERVE | MEM_COMMIT, PAGE_READWRITE); if (!ptr) { - assert("Failed to map virtual memory block" == 0); + assert("Failed to map virtual memory block" && 0); return 0; } #else void* ptr = mmap(0, size + padding, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS | MAP_UNINITIALIZED, -1, 0); if ((ptr == MAP_FAILED) || !ptr) { - assert("Failed to map virtual memory block" == 0); + assert("Failed to map virtual memory block" && 0); return 0; } #endif @@ -1813,12 +1815,12 @@ _memory_unmap_os(void* address, size_t size, size_t offset, int release) { #if PLATFORM_WINDOWS if (!VirtualFree(address, release ? 0 : size, release ? MEM_RELEASE : MEM_DECOMMIT)) { DWORD err = GetLastError(); - assert("Failed to unmap virtual memory block" == 0); + assert("Failed to unmap virtual memory block" && err && 0); } #else MEMORY_UNUSED(release); if (munmap(address, size)) { - assert("Failed to unmap virtual memory block" == 0); + assert("Failed to unmap virtual memory block" && 0); } #endif } @@ -1854,7 +1856,7 @@ _memory_guard_validate(void* p) { if (_memory_config.memory_overwrite) _memory_config.memory_overwrite(p); else - assert("Memory overwrite before block start" == 0); + assert("Memory overwrite before block start" && 0); return; } deadzone[i] = 0; @@ -1866,7 +1868,7 @@ _memory_guard_validate(void* p) { if (_memory_config.memory_overwrite) _memory_config.memory_overwrite(p); else - assert("Memory overwrite after block end" == 0); + assert("Memory overwrite after block end" && 0); return; } deadzone[i] = 0; @@ -2086,4 +2088,8 @@ rpmalloc_global_statistics(rpmalloc_global_statistics_t* stats) { } +#ifdef _MSC_VER +# pragma warning( pop ) +#endif + #endif diff --git a/common/TracySocket.cpp b/common/TracySocket.cpp index 8eb54241..ed14ac08 100644 --- a/common/TracySocket.cpp +++ b/common/TracySocket.cpp @@ -25,6 +25,12 @@ namespace tracy { +#ifdef _MSC_VER +typedef SOCKET socket_t; +#else +typedef int socket_t; +#endif + #ifdef _MSC_VER struct __wsinit { @@ -86,7 +92,7 @@ bool Socket::Connect( const char* addr, const char* port ) hints.ai_socktype = SOCK_STREAM; if( getaddrinfo( addr, port, &hints, &res ) != 0 ) return false; - int sock; + int sock = 0; for( ptr = res; ptr; ptr = ptr->ai_next ) { if( ( sock = socket( ptr->ai_family, ptr->ai_socktype, ptr->ai_protocol ) ) == -1 ) continue; @@ -174,7 +180,7 @@ int Socket::Recv( void* _buf, int len, const timeval* tv ) fd_set fds; FD_ZERO( &fds ); - FD_SET( m_sock, &fds ); + FD_SET( static_cast(m_sock), &fds ); #ifndef _WIN32 timeval _tv = *tv; @@ -231,7 +237,7 @@ bool Socket::HasData() fd_set fds; FD_ZERO( &fds ); - FD_SET( m_sock, &fds ); + FD_SET( static_cast(m_sock), &fds ); return select( m_sock+1, &fds, nullptr, nullptr, &tv ) > 0; } @@ -287,7 +293,7 @@ Socket* ListenSocket::Accept() fd_set fds; FD_ZERO( &fds ); - FD_SET( m_sock, &fds ); + FD_SET( static_cast(m_sock), &fds ); select( m_sock+1, &fds, nullptr, nullptr, &tv ); if( FD_ISSET( m_sock, &fds ) ) diff --git a/common/TracySystem.cpp b/common/TracySystem.cpp index 619c631d..b93353f5 100644 --- a/common/TracySystem.cpp +++ b/common/TracySystem.cpp @@ -1,3 +1,11 @@ +#if defined _MSC_VER || defined __CYGWIN__ || defined _WIN32 +# ifndef WIN32_LEAN_AND_MEAN +# define WIN32_LEAN_AND_MEAN +# endif +# ifndef NOMINMAX +# define NOMINMAX +# endif +#endif #ifdef _WIN32 # include #else diff --git a/common/tracy_lz4.cpp b/common/tracy_lz4.cpp index c6d40ad5..fb2ac3c9 100644 --- a/common/tracy_lz4.cpp +++ b/common/tracy_lz4.cpp @@ -701,7 +701,7 @@ LZ4_FORCE_INLINE int LZ4_compress_generic( cctx->dictSize += (U32)inputSize; } cctx->currentOffset += (U32)inputSize; - cctx->tableType = tableType; + cctx->tableType = (U16)tableType; if (inputSize #include #include + +struct IUnknown; // Workaround for "combaseapi.h(229): error C2187: syntax error: 'identifier' was unexpected here" when using /permissive- #include + #include "nfd_common.h"