Skip to content
This repository has been archived by the owner on Dec 25, 2023. It is now read-only.

clang-cl not used in llvm >= 7.0.0 #7

Closed
ghazel opened this issue Oct 9, 2018 · 3 comments
Closed

clang-cl not used in llvm >= 7.0.0 #7

ghazel opened this issue Oct 9, 2018 · 3 comments

Comments

@ghazel
Copy link

ghazel commented Oct 9, 2018

Changing the version number as in #5 works for 6.0.0, but it does not work for future versions like 7.0.0 (or the 8.0.0 nightlies). When building, MSVC uses cl.exe instead of clang-cl.exe.

What defined clang-cl as the compiler in these scripts?

@ghazel
Copy link
Author

ghazel commented Oct 11, 2018

I see, previous versions had an msbuild-bin directory, with cl.exe (which appears to be an identical copy of clang-cl.exe).

This llvm integration will need to specify cl:

<PropertyGroup>
  <CLToolExe>clang-cl.exe</CLToolExe>
  <CLToolPath>$(LLVMInstallDir)\bin</CLToolPath>
</PropertyGroup>

@arves100
Copy link
Owner

I'm working on making a sort of python script that does everything by default, that's why I haven't answered, btw thanks for the information, I will surely add it (I will close this when the new commit gets pushed)

arves100 added a commit that referenced this issue Dec 29, 2018
@arves100
Copy link
Owner

Thank you for your information, I've pushed a commit under branch v7.0 that adds support for LLVM 7.0.

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

No branches or pull requests

2 participants