-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use native crc32 on x86 if available. #8169
Conversation
Oh, i forgot to see if anyone did this before #4933 |
Hello, thanks for the PR! This looks simpler than #4933.
I would want to tackle this first. |
That would require to change only current crc lookup table |
Changed it, now it produces same results with or without native instruction. |
This is causing problem for people crafting/storing persistent .ini data, which is semi frequently done as a workaround to docking api limitations. I have added a |
…patible with SSE 4.2 instructions. (ocornut#8169, ocornut#4933)
This adds support for the native
_mm_crc32_u8
on x86 provided by the SSE4.2 instruction set (please note that this one actually is crc32c, which uses a different polynomial that is used internally by the Dear ImGui currently).Because it's a native instruction, it doesn't require a 1 kb lookup table.