Remove LLVM dependencies from "common" C++ code


There are a bunch of little dependencies on LLVM within the kythe/cxx/common (e.g., StringRefs), even though nothing says that a language whose indexer is written in C++ necessarily cares about LLVM at all. It would be nice if this dependency could be broken.

mazurak created this task.Via WebJan 21 2016, 12:42 PM
mazurak added a subscriber: mazurak.
schroederc changed the visibility of this Maniphest Task from "All Users" to "Public (No Login Required)".Via WebMay 16 2016, 3:26 PM
schroederc triaged this task as "Low" priority.Via WebMay 17 2016, 9:32 AM
schroederc assigned this task to zarko.Via WebMay 31 2017, 4:09 PM
schroederc added a subscriber: schroederc.

I believe this is fixed, right?

zarko added a comment.Via WebMay 31 2017, 4:14 PM

It's not. The biggest blocker is the lack of std::string_view. We also use some of LLVM's path-manipulation APIs.

shahms added a subscriber: shahms.Via WebWed, Jun 13, 9:33 AM

Bulk reassignment prior to migration, feel free to give to someone else.

Add Comment