Closed NeoAdonis closed 4 months ago
@NeoAdonis.
This PR looks amazing.
Please continue to write down every action you needed to take as you port majority of our samples to support arm64 and observe how that helps.
Looks great. I'd still like to understand the statistics (see above).
@JakobL-MSFT I have added stats to the PR description. As for the next step, there are 4 samples that need fixes before adding support for arm64 to them. These cases will be handled on separate pull requests.
@JakobL-MSFT I have added stats to the PR description. As for the next step, there are 4 samples that need fixes before adding support for arm64 to them. These cases will be handled on separate pull requests.
Great. Thank you for providing all these details!!!
FYI: @GelataioMatto
Why is this change being made?
What changed?
We've made several significant changes to achieve better ARM64 compatibility across the repository:
BufferSecurityCheck
from ARM64 configurations when noEntryPoint
is defined (affects 1 sample)./CETCOMPAT
linker option from ARM64 configurations (affects 1 sample).How was the change tested?
We validated these updates by building all samples targeting ARM64.
Before:
After:
No changes for x64 samples (128 succeeded, 2 excluded for both Debug and Release)