-
Notifications
You must be signed in to change notification settings - Fork 25
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
clangmetatool is too unix centric #35
Comments
|
On Windows, I don't think Clang installs a ClangConfig.cmake file. Version 7.0.1 distribution doesn't include any *.cmake files at all.
|
That would definitely explain the problem... I think we'd need clang changed to support that for us to be able to do this reasonably... |
That's just the first problem. In my experiment, I actually build clang from source and in the install directory created by the install target, the necessary file was present. However, most people don't do this because it takes so long to build clang, so they will likely use an installed package. |
I'm not at all familiar with Windows toolchains, but it seems you're required to build all projects with the same |
I've never done anything where I tried to build a clang tool outside of the clang tree. So far, my excursions with the clang AST/matchers have been in the context of new checks for clang-tidy, e.g. readability-simplify-boolean-expr. |
The description for LNK2038 implies that indeed the runtime library settings used by clang have to match those used by clangmetatool because we're linking against static libraries. (It could be different if it was a shared object.) This could imply that the |
The same is true for iterator debug level; I would have expected the necessary |
Can you please provide your CMakeLists.txt and the commands run? It seems that you are linking the debug version of "mock-api" against the release version of clang/llvm. If the windows build creates both the debug and release versions, then it seems like this isn't really a matter of clangmetatool matching one or the other, but picking the correct version of each in "mock-api". |
Describe the bug
The code only works on Unix and can't be easily setup in Windows.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
It should Just Work.
Screenshots
Environment (please complete the following information):
Additional context
The text was updated successfully, but these errors were encountered: