arphaman added inline comments.
================
Comment at: clangd/tool/ClangdMain.cpp:170
+static llvm::cl::opt<bool>
+ InMemoryCompileCommands("in-memory-compile-commands",
+ llvm::cl::desc("Use an in-memory compile
commands"),
----------------
ilya-biryukov wrote:
> The important distinction seems to be where the compile commands come from,
> maybe make the name centered on the fact that compile commands are read from
> the protocol rather from the filesystem?
>
> E.g. using named option values:
> `--compile_args_from={lsp|filesystem}`
> Or a boolean option:
> `--compile_args_from_lsp`
>
> WDYT?
Yeah, that looks better. I added this argument in the updated patch.
https://reviews.llvm.org/D49758
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
