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

Use clang in VS build? #31456

Open
maflcko opened this issue Dec 10, 2024 · 4 comments
Open

Use clang in VS build? #31456

maflcko opened this issue Dec 10, 2024 · 4 comments

Comments

@maflcko
Copy link
Member

maflcko commented Dec 10, 2024

MSVC has many issues, for example:

Thus, it could make sense to evaluate whether to switch the build docs to clang (possibly with libc++), see https://learn.microsoft.com/en-us/cpp/build/clang-support-msbuild

@willcl-ark
Copy link
Member

Would this open us up to ARM Windows builds as requested in #31388 ?

@ryanofsky
Copy link
Contributor

MSVC has many issues, for example:

Another one:

Am agnostic on this issue though. I could imagine benefits to compiler diversity like ensuring is more portable, or finding bugs missed by other compilers, or allowing more developer tools to be used. But those things might not apply here.

@TheCharlatan
Copy link
Contributor

Also #31158?

@maflcko
Copy link
Member Author

maflcko commented Dec 11, 2024

I could imagine benefits to compiler diversity like ensuring is more portable, or finding bugs missed by other compilers, or allowing more developer tools to be used.

Good point. I guess the only portability issues or bugs were found by using the msSTL, so using clang+msSTL should be fine. If there are real bugs found by msvc (or tools that only work with msvc), it would be good to list them in this brainstorming issue. Otherwise, the reasoning seems recursive, to write portable code to support msvc, for the only reason that msvc can be supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants