-
Notifications
You must be signed in to change notification settings - Fork 123
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
fix cmake dependencies when building together with clang #1168
Conversation
@vgvassilev Compilation succeeded in both cases standalone and along with LLVM. In my local computer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not understand how that change can make a difference but Lgtm.
…)" This reverts commit c1a87d4.
…)" This reverts commit c1a87d4.
…)" This reverts commit c1a87d4.
@Vipul-Cariappa, looks like this did not solve our problem, can you revert it since I think this moves the dependencies in a unexpected place? |
…assilev#1168)" This reverts commit c1a87d4.
@vgvassilev, opened PR at #1173 reverting this. |
…)" This reverts commit c1a87d4.
By any chance is this a better way to describe dependencies?
I am building it locally. Will update you if/when it succeeds.