Conversation
Performance gain was negligible to negative. It also added code complexity and edge cases.
…and storage operations
…ated abstractions
…from Matrix33Base Simplified the Matrix33Base template by removing `SimdOptimal` parameter and all SIMD-related code. Updated related function templates, typedefs, and unit tests to reflect these changes.
…coverage Removed duplicate and unnecessary test cases involving basic matrix addition, subtraction, multiplication, and transposition operations in `Matrix33_UnitTests.cpp` to streamline and align with the simplified `Matrix33Base` implementation.
Implemented 2x2 and 4x4 SIMD matrix inversion functions, including support for SSE2. Added `_mm_swizzle` and `_mm_swizzle4` macros to the SIMD utility header.
…-major abstraction
…nd SIMD configurations for operations
…omprehensive layout and SIMD coverage
…er SIMD performance
… SSE, FMA, and SSE4.1
…nt precision loss
…_ps with _mm_movelh_ps and _mm_movehl_ps
…fines.h> in unit tests
…onConversion unit tests
…ames in VkGraphicsContext
…itwise AND, and comparison
…rol buffer alignment
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
1 participant
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.
Previous implementation added code complexity and edge cases.