Allow detection of CPU arch for GNU/CLang tools #879
Conversation
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.
The macros
_M_AMD64,_M_IX86and_M_ARM64are available into MSVC.However, these macros are not available into GNU tools like MinGW-w64.
This patch doesn't modify any function, it just check which CPU architecture is used on Windows, by also adding the identifiers used by GCC.
To avoid some duplications, the checks into the code are now done by just using:
EDD_DBG_X86_64for AMD64 and Intel x86-64 CPUs.EDD_DBG_X86for 80x86 CPUs.EDD_DBG_ARM64for Aarch64 CPUs.Tested with MinGW-w64 tools with x86_64, i686 and aarch64 cross compiler.