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

VS2019 build doesn't work for x86 #12030

Closed
hoyosjs opened this issue Feb 13, 2019 · 5 comments · Fixed by dotnet/coreclr#22567
Closed

VS2019 build doesn't work for x86 #12030

hoyosjs opened this issue Feb 13, 2019 · 5 comments · Fixed by dotnet/coreclr#22567

Comments

@hoyosjs
Copy link
Member

hoyosjs commented Feb 13, 2019

dotnet/coreclr#22525 Added support to build using VS2019. However, there seems to be a bug in the cmake generator for VS2019. Specifying x86 as the architecture still uses the x64 variant of CL in the generator and the step that tries to consume the generated vcxproj files fails - at least locally. All the generated files seem to target x64 as well. I've opened an issue against cmake to see if there is a workaround.

@AaronRobinsonMSFT
Copy link
Member

@RussKeldorph Unsure what our timeline for moving to the latest VS, but this might be something to keep an eye on.

@tannergooding
Copy link
Member

This might be a case where we should explicitly specify -A Win32. Right now we are just leaving it up to the CMake default.

@tannergooding
Copy link
Member

CC. @safern as this might be a concern for CoreFX as well.

@tannergooding
Copy link
Member

tannergooding commented Feb 13, 2019

@hoyosjs, could you try adding the following line to https://github.com/dotnet/coreclr/blob/master/src/pal/tools/gen-buildsys-win.bat#L29 and seeing if that fixes your issue?

if /i "%__Arch%" == "x86" (set __ExtraCmakeParams=%__ExtraCmakeParams% -A Win32)

(if not, that is fine, and I will test in the morning)

@hoyosjs
Copy link
Member Author

hoyosjs commented Feb 13, 2019

This seems to fix the issue indeed. I'll make the PR now. Thanks!

@msftgits msftgits transferred this issue from dotnet/coreclr Jan 31, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants