externpro 26.01.1-49-g8c08f0b#5
Merged
Merged
Conversation
- Update encode() to use contiguous input and callback-based output - Update decode() to use std::map and callback-based output - Add temporary buffers to prevent share data corruption during decode - Replace deprecated sprintf() with snprintf() - Fix format specifiers for size_t types - Add missing vector include and fecpp namespace usage
- Conditionally include fecpp_sse2.cpp and fecpp_ssse3.cpp on x86_64 platforms - Test compiler support for SSE2/SSSE3 intrinsics before inclusion - Apply appropriate compile flags (-msse2, -mssse3) for GCC/Clang - Handle MSVC x64 build with automatic SSE2 and conditional SSSE3 support - Exclude SIMD sources on non-x86 architectures (ARM64, etc.)
- Add cpuid.cpp to main library sources for universal CPU detection - Replace weak has_sse2()/has_ssse3() implementations with comprehensive detection - Support MSVC x64 (always SSE2), MSVC x86, GCC/Clang runtime detection, and fallbacks - Move intrin.h include inside MSVC-specific blocks for cross-platform compatibility - Enable future ARM64 support by centralizing CPU capability detection
- Add intrin.h include for MSVC intrinsics support - Replace __builtin_clz with _BitScanReverse for MSVC compatibility - Fix _mm_prefetch calls with explicit const char* casts for MSVC - Maintain GCC/Clang compatibility through conditional compilation
- Wrap intrin.h include in MSVC conditional to fix Linux AMD64 builds - Expand Windows detection to include _WIN32/_WIN64 for modern Windows compilers - Fix sys/time.h inclusion error on Windows CI systems - Maintain compatibility across Windows MSVC, Linux GCC/Clang, and macOS
- Extend Windows detection to include _WIN32/_WIN64 for modern MSVC compilers - Add Windows-specific TICK macro using ticks instead of tv_sec/tv_usec - Fix bcmp function availability on Windows CI systems - Maintain original code structure with minimal targeted changes
- Move #endif to include Unix TICK macro definition - Add TICK macro to Windows block for proper platform separation - Prevent macro redefinition warning on Windows CI builds - Maintain original macro ordering and structure
Collaborator
Author
|
issue externpro/externpro#207 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Update externpro submodule to
26.01.1-49-g8c08f0bChanges
.devcontainerto externpro26.01.1-49-g8c08f0b0e352b953871074477bdddbd9cc0cf1b5c4d6b9d8c08f0bbbcff882a932fd81552aa161a829fff15This PR was created automatically by GitHub Actions