From 37d5736bf51592a6d803576dc34b7062edc2a1d6 Mon Sep 17 00:00:00 2001 From: Till Rathmann Date: Wed, 1 Aug 2018 14:07:30 +0200 Subject: [PATCH] Fixed compiler warnings. --- TracyClient.cpp | 4 ---- client/TracyCallstack.cpp | 3 +++ client/TracyProfiler.cpp | 7 ++++--- client/TracyProfiler.hpp | 2 +- client/concurrentqueue.h | 29 ++++++++++++++++++++++------- client/tracy_rpmalloc.cpp | 18 ++++++++++++------ common/TracySocket.cpp | 8 ++++---- common/TracySystem.cpp | 8 ++++++++ common/tracy_lz4.cpp | 2 +- common/tracy_lz4.hpp | 2 +- 10 files changed, 56 insertions(+), 27 deletions(-) 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..1f41d35a 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 ); } @@ -139,8 +139,9 @@ static const char* GetProcessName() # endif #elif defined _GNU_SOURCE || defined __CYGWIN__ return program_invocation_short_name; -#endif +#else return "unknown"; +#endif } 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..bb958358 100644 --- a/common/TracySocket.cpp +++ b/common/TracySocket.cpp @@ -86,7 +86,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 +174,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 +231,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 +287,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