Skip to content
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

Fix ARM compilation for windows. #401

Merged
merged 1 commit into from
Apr 8, 2021

Conversation

englercj
Copy link
Contributor

@englercj englercj commented Mar 7, 2021

Added support for ARM in the windows stacktrace utility.

@KjellKod
Copy link
Owner

KjellKod commented Mar 7, 2021

Ideally this would be covered by one of the cloud CIs. I don’t see that as feasible so I’ll approve and merge it.

Please think about it, if it’s available with ARM on any of the 3 cloud CI we are using them adding ARM support for it would help to future proof ARM support

@KjellKod
Copy link
Owner

KjellKod commented Mar 7, 2021

I’m holding off with merging until you you give me the OK that you are confident it’ll work for ARM.

I’ve no way of verifying this apart from code reviewing

@englercj
Copy link
Contributor Author

englercj commented Mar 8, 2021

Unfortunately I too don't have a machine to verify on, yet. Also unfortunate is the complexity of windows ARM where you generally build on x86 and deploy to ARM.

I've seen similar changes to other stack trace handling code that was updated for ARM though so I'm fairly confident this is correct. I'm also fine leaving it open until we have a chance to verify, but it may be some time for me before I get there. I'm building for ARM64 windows currently, but not ready for testing yet.

@englercj
Copy link
Contributor Author

englercj commented Mar 8, 2021

I think AppVeyor has ARM64 support (from googling) but I'm not familiar enough with that platform you get it running.

@KjellKod
Copy link
Owner

KjellKod commented Apr 8, 2021

I'm going out on a limb here and making it available. Please consider adding CI support for this later

@KjellKod KjellKod merged commit d3f2d11 into KjellKod:master Apr 8, 2021
@englercj englercj deleted the cengler/arm-fix branch July 27, 2021 20:26
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