Skip to content

Add VS17 configuration + ARM64#13

Merged
cmb69 merged 3 commits intowinlibs:masterfrom
php-win-ext:master
Oct 27, 2024
Merged

Add VS17 configuration + ARM64#13
cmb69 merged 3 commits intowinlibs:masterfrom
php-win-ext:master

Conversation

@macintoshplus
Copy link
Copy Markdown
Contributor

@macintoshplus macintoshplus commented Sep 6, 2024

I have updated the project for Visual Studio 17 2022 and added the ARM64 configuration.

On ARM64 configuration, I have configured the OutDir with the same valeu from x64 and added the \bin or \lib folder.
To fix the linker warning MSB8012, I have removed the \bin or \lib folder on the output file configuration.

I have also enabled DYNAMICBASE to solve LINK : fatal error LNK1246: '/DYNAMICBASE:NO' not compatible with 'ARM64' target machine; link without '/DYNAMICBASE:NO'

@cmb69
Copy link
Copy Markdown
Member

cmb69 commented Oct 27, 2024

Thank you! This looks good to me.

Note that there is PR #11 which added ARM64 support for MSVC16, likely somewhat differently. In my opinion, it is a bad idea to have separate solutions for different VS versions; at least nowadays there shouldn't be the need to have this. Still, for BC it might be best to stick with this for now.

@cmb69 cmb69 merged commit d1ea8f7 into winlibs:master Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants