-
Notifications
You must be signed in to change notification settings - Fork 244
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
Remove main repo dependencies on LLVM #112
Milestone
Comments
zrlk
added a commit
to zrlk/kythe
that referenced
this issue
Jun 4, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses kythe#112.
zrlk
added a commit
to zrlk/kythe
that referenced
this issue
Jun 4, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses kythe#112.
zrlk
added a commit
to zrlk/kythe
that referenced
this issue
Jun 4, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses kythe#112.
zrlk
added a commit
to zrlk/kythe
that referenced
this issue
Jun 4, 2018
This utility file is specific to C++ and is used only by the C++ extractor. Partially addresses kythe#112.
zrlk
added a commit
that referenced
this issue
Jun 5, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses #112.
zrlk
added a commit
that referenced
this issue
Jun 5, 2018
This utility file is specific to C++ and is used only by the C++ extractor. Partially addresses #112.
schroederc
pushed a commit
to schroederc/kythe
that referenced
this issue
Nov 8, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses kythe#112.
schroederc
pushed a commit
to schroederc/kythe
that referenced
this issue
Nov 8, 2018
This utility file is specific to C++ and is used only by the C++ extractor. Partially addresses kythe#112.
schroederc
pushed a commit
to schroederc/kythe
that referenced
this issue
Nov 8, 2018
The VFS interface is LLVM-specific and shouldn't be visible external to the C++ indexer. Partially addresses kythe#112.
schroederc
pushed a commit
to schroederc/kythe
that referenced
this issue
Nov 8, 2018
This utility file is specific to C++ and is used only by the C++ extractor. Partially addresses kythe#112.
I believe this may be almost done:
Note that (almost?) all the direct LLVM dependencies are now the C++ indexer or extractor (which necessarily requires LLVM), or LLVM Is there anything further we need to address here? |
As far as I can tell, this is done. Feel free to reopen it if I missed something. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now the main Kythe repo depends on LLVM in ways other than just the C++ indexer itself. It's fine for the C++ indexer to depend on LLVM, but the rest of the main repository shouldn't.
As far as I know the only reason we depend on it is for some library components, and we may be able to get those from Abseil instead.
The text was updated successfully, but these errors were encountered: