[clang-tools-extra] r344614 - Remove possibility to change compile database path at runtime

2018-10-16 Thread Simon Marchi via cfe-commits
Author: simark Date: Tue Oct 16 08:55:03 2018 New Revision: 344614 URL: http://llvm.org/viewvc/llvm-project?rev=344614=rev Log: Remove possibility to change compile database path at runtime Summary: This patch removes the possibility to change the compilation database path at runtime using the

[clang-tools-extra] r339483 - [clangd] Avoid duplicates in findDefinitions response

2018-08-10 Thread Simon Marchi via cfe-commits
Author: simark Date: Fri Aug 10 15:27:53 2018 New Revision: 339483 URL: http://llvm.org/viewvc/llvm-project?rev=339483=rev Log: [clangd] Avoid duplicates in findDefinitions response Summary: When compile_commands.json contains some source files expressed as relative paths, we can get duplicate

r339063 - [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name

2018-08-06 Thread Simon Marchi via cfe-commits
Author: simark Date: Mon Aug 6 14:48:20 2018 New Revision: 339063 URL: http://llvm.org/viewvc/llvm-project?rev=339063=rev Log: [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name Summary: InMemoryFileSystem::status behaves differently than

[clang-tools-extra] r338914 - [clangd] Add test for changing build configuration

2018-08-03 Thread Simon Marchi via cfe-commits
Author: simark Date: Fri Aug 3 12:40:19 2018 New Revision: 338914 URL: http://llvm.org/viewvc/llvm-project?rev=338914=rev Log: [clangd] Add test for changing build configuration Summary: This patch adds tests for the two ways of changing build configuration (pointing to a particular

Re: [clang-tools-extra] r338518 - [clangd] Receive compilationDatabasePath in 'initialize' request

2018-08-01 Thread Simon Marchi via cfe-commits
On 2018-08-01 01:30 PM, Alex L wrote: > Is there a particular reason why this commit didn't have a corresponding test > included? > Cheers, > Alex Back when we made the corresponding change in "onChangeConfiguration", there was no straightforward way to make a lit test for it:

Re: [clang-tools-extra] r338518 - [clangd] Receive compilationDatabasePath in 'initialize' request

2018-08-01 Thread Simon Marchi via cfe-commits
On 2018-08-01 01:30 PM, Alex L wrote: > Is there a particular reason why this commit didn't have a corresponding test > included? > Cheers, > Alex Back when we made the corresponding change in "onChangeConfiguration", there was no straightforward way to make a lit test for it:

[clang-tools-extra] r338518 - [clangd] Receive compilationDatabasePath in 'initialize' request

2018-08-01 Thread Simon Marchi via cfe-commits
Author: simark Date: Wed Aug 1 04:28:49 2018 New Revision: 338518 URL: http://llvm.org/viewvc/llvm-project?rev=338518=rev Log: [clangd] Receive compilationDatabasePath in 'initialize' request Summary: That way, as soon as the "initialize" is received by the server, it can start parsing/indexing

r338057 - [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name

2018-07-26 Thread Simon Marchi via cfe-commits
Author: simark Date: Thu Jul 26 11:55:02 2018 New Revision: 338057 URL: http://llvm.org/viewvc/llvm-project?rev=338057=rev Log: [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name Summary: InMemoryFileSystem::status behaves differently than

[clang-tools-extra] r337697 - [clangd] Fix category in clangd-vscode's package.json

2018-07-23 Thread Simon Marchi via cfe-commits
Author: simark Date: Mon Jul 23 07:32:12 2018 New Revision: 337697 URL: http://llvm.org/viewvc/llvm-project?rev=337697=rev Log: [clangd] Fix category in clangd-vscode's package.json Summary: When opening package.json, vscode shows: Use 'Programming Languages' instead Replacing "Languages"

r337284 - [Tooling] Add operator== to CompileCommand

2018-07-17 Thread Simon Marchi via cfe-commits
Author: simark Date: Tue Jul 17 07:13:05 2018 New Revision: 337284 URL: http://llvm.org/viewvc/llvm-project?rev=337284=rev Log: [Tooling] Add operator== to CompileCommand Summary: It does the obvious thing of comparing all fields. This will be needed for a clangd patch I have in the pipeline.

r336807 - [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name

2018-07-11 Thread Simon Marchi via cfe-commits
Author: simark Date: Wed Jul 11 07:08:17 2018 New Revision: 336807 URL: http://llvm.org/viewvc/llvm-project?rev=336807=rev Log: [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name Summary: InMemoryFileSystem::status behaves differently than

[clang-tools-extra] r336358 - [clang-move] ClangMoveTests: Remove dots in output paths

2018-07-05 Thread Simon Marchi via cfe-commits
Author: simark Date: Thu Jul 5 07:53:17 2018 New Revision: 336358 URL: http://llvm.org/viewvc/llvm-project?rev=336358=rev Log: [clang-move] ClangMoveTests: Remove dots in output paths Summary: Following D48903 ([VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested

Re: [clang-tools-extra] r328500 - [clangd] Support incremental document syncing

2018-03-28 Thread Simon Marchi via cfe-commits
On 2018-03-27 01:47 PM, Reid Kleckner wrote: > One of these new tests does not pass on Windows due to assumptions about > absolute path structure. The FileCheck output seems self-explanatory: > > $ "FileCheck" "-strict-whitespace" >

[clang-tools-extra] r328500 - [clangd] Support incremental document syncing

2018-03-26 Thread Simon Marchi via cfe-commits
Author: simark Date: Mon Mar 26 07:41:40 2018 New Revision: 328500 URL: http://llvm.org/viewvc/llvm-project?rev=328500=rev Log: [clangd] Support incremental document syncing Summary: This patch adds support for incremental document syncing, as described in the LSP spec. The protocol specifies

[clang-tools-extra] r328100 - Make positionToOffset return llvm::Expected

2018-03-21 Thread Simon Marchi via cfe-commits
Author: simark Date: Wed Mar 21 07:36:46 2018 New Revision: 328100 URL: http://llvm.org/viewvc/llvm-project?rev=328100=rev Log: Make positionToOffset return llvm::Expected Summary: To implement incremental document syncing, we want to verify that the ranges provided by the front-end are valid.

[clang-tools-extra] r327711 - Move DraftMgr from ClangdServer to ClangdLSPServer

2018-03-16 Thread Simon Marchi via cfe-commits
Author: simark Date: Fri Mar 16 07:30:42 2018 New Revision: 327711 URL: http://llvm.org/viewvc/llvm-project?rev=327711=rev Log: Move DraftMgr from ClangdServer to ClangdLSPServer Summary: This patch moves the draft manager closer to the edge of Clangd, from ClangdServer to ClangdLSPServer. This

[clang-tools-extra] r327550 - [clangd] Use Contents from inputs in codeComplete and signatureHelp

2018-03-14 Thread Simon Marchi via cfe-commits
Author: simark Date: Wed Mar 14 11:31:48 2018 New Revision: 327550 URL: http://llvm.org/viewvc/llvm-project?rev=327550=rev Log: [clangd] Use Contents from inputs in codeComplete and signatureHelp Summary: ClangdServer::{codeComplete,signatureHelp} both use the Contents from the draft manager.

[clang-tools-extra] r325784 - [clangd] DidChangeConfiguration Notification

2018-02-22 Thread Simon Marchi via cfe-commits
Author: simark Date: Thu Feb 22 06:00:39 2018 New Revision: 325784 URL: http://llvm.org/viewvc/llvm-project?rev=325784=rev Log: [clangd] DidChangeConfiguration Notification Summary: Implementation of DidChangeConfiguration notification handling in clangd. This currently only supports changing

[clang-tools-extra] r325596 - [clangd] Fix formatting in XRefs.cpp

2018-02-20 Thread Simon Marchi via cfe-commits
Author: simark Date: Tue Feb 20 08:57:47 2018 New Revision: 325596 URL: http://llvm.org/viewvc/llvm-project?rev=325596=rev Log: [clangd] Fix formatting in XRefs.cpp This is also to test my commit access. Modified: clang-tools-extra/trunk/clangd/XRefs.cpp Modified: