carbon-lang
carbon-lang copied to clipboard
Remove clangd BUILD patch
#3112 patched the LLVM build to expose files from clangd, here. If possible, this should be upstreamed: we should be trying to stay as close to LLVM upstream as we can, avoiding patches which we would need to maintain.
If it can't be upstreamed because upstream doesn't want these files used as a library, the dependency should be re-evaluated. In general we don't want to add dependencies on internal libraries, because of a lack of long-term stability guarantees.
FYI @josh11b @Maan2003
We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please comment or remove the inactive label. The long term label can also be added for issues which are expected to take time.
This issue is labeled inactive because the last activity was over 90 days ago.
@josh11b Is this something that you can investigate? (I'm concerned about what happens if there's an API change, or the patch breaks)
Got https://github.com/llvm/llvm-project/pull/81556 accepted upstream. Next time we sync llvm, we can hopefully remove (most of?) the patch (though the library did get renamed in the process).
Would you like to take care of syncing?
next time we sync llvm ,