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

[Invalid] Try building with ClangCL #1407

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Neumann-A
Copy link

Try reproducing #1405 in VSG own CI (ClangCL toolset should be installed in the default windows runners if i remember correctly.)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@Neumann-A
Copy link
Author

Feel free to close if you don't need those changes here.

@robertosfield
Copy link
Collaborator

Thanks for the CL change.

I have started looking at the log. Lots of the warnings about incompatibility are really brain dead. The VSG build system explicitly specifies C++17 so not being compatible with C++98 and C++14 is exactly what is expected, so the compiler warnings are just wrong. I have seen anyone else report these warnings so perhaps just something about the clang version or default warnings that area enabled.

I'm away on a trip and the internet connections here is bad so it's taking forever to browse through the logs so haven't been able to browse through to the actual compile error. I'll leave this Draft PR up till me return to office next week.

What version of clang are you using? I've previously built with clang under Linux (that's my main dev platform) without issues.

@Neumann-A
Copy link
Author

The error is posted here: #1405 (comment) it is the same error as I have observed. Your VS version/CI uses 18.1.x, I use 19.1.x

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.

None yet

2 participants