Update Optimizer (float arithmetic and more) #472
There are no files selected for viewing
This file contains 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
Binary file not shown.
This file contains 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
This file contains 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
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,100 @@ | ||
| #include "cpuinfo.h" | ||
|
|
||
|
|
||
| #ifdef _WIN32 | ||
|
|
||
| // Windows | ||
| #include <intrin.h> | ||
| #define cpuid(info, x) __cpuidex(info, x, 0) | ||
|
|
||
| #else | ||
|
|
||
| // GCC Intrinsics | ||
| #include <cpuid.h> | ||
| void cpuid(int info[4], int InfoType){ | ||
| __cpuid_count(InfoType, 0, info[0], info[1], info[2], info[3]); | ||
| } | ||
|
|
||
| #endif | ||
|
|
||
| // CPU Info | ||
| CPUInfo::CPUInfo() | ||
| { | ||
| int info[4]; | ||
| cpuid(info, 0); | ||
| int nIds = info[0]; | ||
|
|
||
| cpuid(info, 0x80000000); | ||
| unsigned nExIds = info[0]; | ||
|
|
||
| // Detect Features | ||
| if (nIds >= 0x00000001) | ||
| { | ||
| cpuid(info,0x00000001); | ||
| m_has_cmov = (info[3] & ((int)1 << 15)) != 0; | ||
| m_has_mmx = (info[3] & ((int)1 << 23)) != 0; | ||
|
|
||
| m_has_sse = (info[3] & ((int)1 << 25)) != 0; | ||
| m_has_sse2 = (info[3] & ((int)1 << 26)) != 0; | ||
| m_has_sse3 = (info[2] & ((int)1 << 0)) != 0; | ||
| m_has_ssse3 = (info[2] & ((int)1 << 9)) != 0; | ||
| m_has_sse4_1 = (info[2] & ((int)1 << 19)) != 0; | ||
| m_has_sse4_2 = (info[2] & ((int)1 << 20)) != 0; | ||
|
|
||
| m_has_clmul = (info[2] & ((int)1 << 1)) != 0; | ||
| m_has_fma3 = (info[2] & ((int)1 << 12)) != 0; | ||
| m_has_aes = (info[2] & ((int)1 << 25)) != 0; | ||
| m_has_avx = (info[2] & ((int)1 << 28)) != 0; | ||
| m_has_f16c = (info[2] & ((int)1 << 29)) != 0; | ||
| m_has_rdrand = (info[2] & ((int)1 << 30)) != 0; | ||
| } | ||
| if (nIds >= 0x00000007) | ||
| { | ||
| cpuid(info,0x00000007); | ||
|
|
||
| m_has_avx2 = (info[1] & ((int)1 << 5)) != 0; | ||
| m_has_avx512F = (info[1] & ((int)1 << 16)) != 0; | ||
| m_has_avx512CD = (info[1] & ((int)1 << 28)) != 0; | ||
| m_has_avx512PF = (info[1] & ((int)1 << 26)) != 0; | ||
| m_has_avx512ER = (info[1] & ((int)1 << 27)) != 0; | ||
| m_has_avx512VL = (info[1] & ((int)1 << 31)) != 0; | ||
| m_has_avx512BW = (info[1] & ((int)1 << 30)) != 0; | ||
| m_has_avx512DQ = (info[1] & ((int)1 << 17)) != 0; | ||
| m_has_avx512IFMA = (info[1] & ((int)1 << 21)) != 0; | ||
| m_has_avx512VBMI = (info[2] & ((int)1 << 1)) != 0; | ||
| } | ||
| if (nExIds >= 0x80000001) | ||
| { | ||
| cpuid(info,0x80000001); | ||
|
|
||
| m_has_xop = (info[2] & ((int)1 << 11)) != 0; | ||
| } | ||
| } | ||
|
|
||
|
|
||
| bool CPUInfo::has_cmov() const { return m_has_cmov; } | ||
| bool CPUInfo::has_mmx() const { return m_has_mmx; } | ||
| bool CPUInfo::has_sse() const { return m_has_sse; } | ||
| bool CPUInfo::has_sse2() const { return m_has_sse2; } | ||
| bool CPUInfo::has_sse3() const { return m_has_sse3; } | ||
| bool CPUInfo::has_ssse3() const { return m_has_ssse3; } | ||
| bool CPUInfo::has_sse4_1() const { return m_has_sse4_1; } | ||
| bool CPUInfo::has_sse4_2() const { return m_has_sse4_2; } | ||
| bool CPUInfo::has_clmul() const { return m_has_clmul; } | ||
| bool CPUInfo::has_fma3() const { return m_has_fma3; } | ||
| bool CPUInfo::has_aes() const { return m_has_aes; } | ||
| bool CPUInfo::has_avx() const { return m_has_avx; } | ||
| bool CPUInfo::has_avx2() const { return m_has_avx2; } | ||
| bool CPUInfo::has_f16c() const { return m_has_f16c; } | ||
| bool CPUInfo::has_rdrand() const { return m_has_rdrand; } | ||
| bool CPUInfo::has_xop() const { return m_has_xop; } | ||
| bool CPUInfo::has_avx512F() const { return m_has_avx512F; } | ||
| bool CPUInfo::has_avx512CD() const { return m_has_avx512CD; } | ||
| bool CPUInfo::has_avx512PF() const { return m_has_avx512PF; } | ||
| bool CPUInfo::has_avx512ER() const { return m_has_avx512ER; } | ||
| bool CPUInfo::has_avx512VL() const { return m_has_avx512VL; } | ||
| bool CPUInfo::has_avx512BW() const { return m_has_avx512BW; } | ||
| bool CPUInfo::has_avx512DQ() const { return m_has_avx512DQ; } | ||
| bool CPUInfo::has_avx512IFMA() const { return m_has_avx512IFMA; } | ||
| bool CPUInfo::has_avx512VBMI() const { return m_has_avx512VBMI; } | ||
|
|
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,70 @@ | ||
| #ifndef _INCLUDE_AMXMODX_CPUINFO_H | ||
| #define _INCLUDE_AMXMODX_CPUINFO_H | ||
|
|
||
| class CPUInfo | ||
| { | ||
| private: | ||
| bool m_has_cmov; | ||
| bool m_has_mmx; | ||
|
|
||
| bool m_has_sse; | ||
| bool m_has_sse2; | ||
| bool m_has_sse3; | ||
| bool m_has_ssse3; | ||
| bool m_has_sse4_1; | ||
| bool m_has_sse4_2; | ||
|
|
||
| bool m_has_clmul; | ||
| bool m_has_fma3; | ||
| bool m_has_aes; | ||
| bool m_has_avx; | ||
| bool m_has_avx2; | ||
| bool m_has_f16c; | ||
| bool m_has_rdrand; | ||
| bool m_has_xop; | ||
|
|
||
| bool m_has_avx512F; | ||
| bool m_has_avx512CD; | ||
| bool m_has_avx512PF; | ||
| bool m_has_avx512ER; | ||
| bool m_has_avx512VL; | ||
| bool m_has_avx512BW; | ||
| bool m_has_avx512DQ; | ||
| bool m_has_avx512IFMA; | ||
| bool m_has_avx512VBMI; | ||
|
|
||
| public: | ||
| CPUInfo(); | ||
|
|
||
| bool has_cmov() const; | ||
| bool has_mmx() const; | ||
|
|
||
| bool has_sse() const; | ||
| bool has_sse2() const; | ||
| bool has_sse3() const; | ||
| bool has_ssse3() const; | ||
| bool has_sse4_1() const; | ||
| bool has_sse4_2() const; | ||
|
|
||
| bool has_clmul() const; | ||
| bool has_fma3() const; | ||
| bool has_aes() const; | ||
| bool has_avx() const; | ||
| bool has_avx2() const; | ||
| bool has_f16c() const; | ||
| bool has_rdrand() const; | ||
| bool has_xop() const; | ||
|
|
||
| bool has_avx512F() const; | ||
| bool has_avx512CD() const; | ||
| bool has_avx512PF() const; | ||
| bool has_avx512ER() const; | ||
| bool has_avx512VL() const; | ||
| bool has_avx512BW() const; | ||
| bool has_avx512DQ() const; | ||
| bool has_avx512IFMA() const; | ||
| bool has_avx512VBMI() const; | ||
| }; | ||
|
|
||
|
|
||
| #endif //_INCLUDE_AMXMODX_CPUINFO_H |
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is correct but breaks compatibility. Oddly, amxmodx doesn't do regular "banker's rounding", but instead does basically
floor(x + 0.5). Therefore, without this change, rounding 2.5 gives you 3, and with your change rounding 2.5 gives you 2.Personally, I wouldn't mind breaking compatibility here, but considering this could easily break some plugins, Arkshine and others might not agree with this change, and they'd probably be right.
See this PR for some extra info.